Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Authorities on custom_operation are not checked correctly #649

Closed
nathanielhourt opened this issue May 9, 2016 · 1 comment
Closed

Authorities on custom_operation are not checked correctly #649

nathanielhourt opened this issue May 9, 2016 · 1 comment

Comments

@nathanielhourt
Copy link
Contributor

The required_auths field on custom_operation should require extra signatures for the accounts listed, but it does not:
https://bitshares.openledger.info/#/block/5975061
nathanielhourt added a commit to FollowMyVote/graphene that referenced this issue May 25, 2016
@nathanielhourt
Resolve cryptonomex#649: [HF] Check authorities on custom_operation
373700e 
The required_auths field on custom_operation was being ignored during authority checking. This commit causes it to be checked correctly, and adds a unit test verifying as much.
nathanielhourt added a commit to FollowMyVote/graphene that referenced this issue May 25, 2016
@nathanielhourt
Resolve cryptonomex#649: [HF] Check authorities on custom_operation
0bb157e 
The required_auths field on custom_operation was being ignored during authority checking. This commit causes it to be checked correctly, and adds a unit test verifying as much.
@vikramrajkumar vikramrajkumar mentioned this issue Jan 18, 2017
Closed
@vikramrajkumar
Copy link
Contributor

This issue was moved to bitshares/bitshares-core#210

pmconrad pushed a commit to pmconrad/graphene that referenced this issue Apr 17, 2018
@abitmore
Test case reproduces cryptonomex#338 cryptonomex#343 cryptonomex#453 c…
10298ca 
…ryptonomex#606 cryptonomex#625 cryptonomex#649
pmconrad pushed a commit to pmconrad/graphene that referenced this issue Apr 17, 2018
@abitmore
Test case for cryptonomex#338 cryptonomex#343 cryptonomex#453 crypton…
e5fc5d4 
…omex#606 cryptonomex#625 cryptonomex#649 fixes
pmconrad pushed a commit to pmconrad/graphene that referenced this issue Apr 17, 2018
@abitmore
Add cross-hardfork tests cryptonomex#338 cryptonomex#343 cryptonomex#453
a8391d8 
 cryptonomex#606 cryptonomex#649
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nathanielhourt @vikramrajkumar