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.

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

inconsistent error message for update_asset_issuer #944

Closed
tianyekuo opened this issue May 22, 2018 · 3 comments

Comments

Projects
3 participants
@tianyekuo
Copy link

commented May 22, 2018

Issues:
For the new feature BSIP29, if only import activity key of the asset issuer without import the owner key, then call the API update_asset_issuer. It will fail with this message Missing Active Authority 1.2.22628
In fact, the activity key is provided, user may get confused here.

Expect:
It should warning Missing Owner Authority.

@abitmore

This comment has been minimized.

Copy link
Member

commented May 22, 2018

I'll look into this.

Note: this is a minor issue, but perhaps hard to fix, so probably unable to fix in this release.

@abitmore

This comment has been minimized.

Copy link
Member

commented May 22, 2018

I believe #580 is the other side of the same issue. The reason is: active authority of fee_payer is checked first, even when the operation requires owner authority of same account.

@abitmore abitmore modified the milestones: 201806 - Consensus Changing Release, Future Non-Consensus-Changing Release May 22, 2018

@ryanRfox ryanRfox added this to New -Awaiting Core Team Evaluation in Project Backlog via automation May 23, 2018

@ryanRfox ryanRfox moved this from New -Awaiting Core Team Evaluation to Unassigned - Bugs in Project Backlog May 23, 2018

abitmore added a commit that referenced this issue Aug 14, 2018

Added test case to reproduce #944
tx_missing_active_auth exception is thrown when owner authority is missing

abitmore added a commit that referenced this issue Aug 14, 2018

@abitmore abitmore added this to To do in Feature release (201810) via automation Aug 14, 2018

@abitmore abitmore removed this from Unassigned - Bugs in Project Backlog Aug 14, 2018

@abitmore abitmore moved this from To do to In progress in Feature release (201810) Aug 14, 2018

abitmore added a commit that referenced this issue Aug 16, 2018

Merge pull request #1255 from bitshares/944-missing-owner-auth
Fix #944 unexpected "Missing Active Authority" message for update_asset_issuer
@abitmore

This comment has been minimized.

Copy link
Member

commented Aug 16, 2018

Fixed by #1255.

@abitmore abitmore closed this Aug 16, 2018

Feature release (201810) automation moved this from In progress to Done Aug 16, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.