feat(standards): bound the fee payment of the plain multisig component - #3802
Open
mmagician wants to merge 2 commits into
Open
feat(standards): bound the fee payment of the plain multisig component#3802mmagician wants to merge 2 commits into
mmagician wants to merge 2 commits into
Conversation
…ig components The guarded and smart multisig components carried the same bounded fee flow; it now lives in `multisig::pay_bounded_fee`, with the bound defined once, and the components keep only what differs (the guardian signature slot). Inline comments trimmed as requested on #3798. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KkQzFtRjyDwQ7iVtRbsHsn
`AuthMultisig` paid the fee through the unbounded `fee::pay_fee`, so a host-supplied rate could move the vault's fee-asset balance into the TX_FEE note, below the default quorum where a per-procedure override applies (#3763). It now pays through `multisig::pay_bounded_fee` like the guarded and smart components: native fee asset only, at most twice the computed fee. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KkQzFtRjyDwQ7iVtRbsHsn
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Closes #3763.
#3786 bounded the fee payment of the guarded and smart multisig component, but left plain
AuthMultisigunbounded, The same change lands onnextas part of #3798.