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

Unable to marker transfer from a group account #1818

Closed
4 tasks
jtalis opened this issue Jan 18, 2024 · 1 comment · Fixed by #1821
Closed
4 tasks

Unable to marker transfer from a group account #1818

jtalis opened this issue Jan 18, 2024 · 1 comment · Fixed by #1821
Assignees

Comments

@jtalis
Copy link
Contributor

jtalis commented Jan 18, 2024

Summary of Bug

When a marker has allow_forced_transfer set to true, a marker transfer from a group account fails for an account with MARKER_TRANSFER permission even through an authz grant exists

Version

1.17.1

Steps to Reproduce

  1. Create a restricted marker with allow_forced_transfer = true and withdraw some coin to a group account.
  2. Create an authz grant for the group account to allow transfer of the marker by an account with MARKER_TRANSFER permissions
  3. Signing with the account with marker transfer permissions, attempt to transfer the coin from the group account.
  4. Note the error funds are not allowed to be removed from {group address}

example failure:

https://explorer.test.provenance.io/tx/3740C759C9A3117E84E4B6B9B9273B2AE351CC0DEC7CC54191DDD50B8C90D454/19980649


For Admin Use

  • Not duplicate issue
  • Appropriate labels applied
  • Appropriate contributors tagged
  • Contributor assigned/self-assigned
@Taztingo Taztingo self-assigned this Jan 24, 2024
@vwagner
Copy link

vwagner commented Jan 26, 2024

Just a comment that this transaction is slightly odd in that the smart contract (the grantee of the transfer authority authz grant) already had TRANSFER authority on the Marker (with force transfer enabled). In other words, the authz grant here was likely unnecessary.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants