Skip to content

Cap max fee at 5% to prevent collateral rug via fee manipulation#3

Merged
anderdc merged 1 commit intotestfrom
fee-cap-5pct
Mar 26, 2026
Merged

Cap max fee at 5% to prevent collateral rug via fee manipulation#3
anderdc merged 1 commit intotestfrom
fee-cap-5pct

Conversation

@LandynDev
Copy link
Copy Markdown
Collaborator

Previously fee_divisor minimum was 2 (allowing up to 50% fee on miner collateral per swap). Now enforced at minimum 20 (max 5%) in both the contract and CLI.

Previously fee_divisor minimum was 2 (allowing up to 50% fee on miner
collateral per swap). Now enforced at minimum 20 (max 5%) in both the
contract and CLI.
Copy link
Copy Markdown
Collaborator

@anderdc anderdc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@anderdc anderdc merged commit eeb3313 into test Mar 26, 2026
2 checks passed
@anderdc anderdc deleted the fee-cap-5pct branch March 26, 2026 01:59
anderdc pushed a commit that referenced this pull request Mar 26, 2026
Previously fee_divisor minimum was 2 (allowing up to 50% fee on miner
collateral per swap). Now enforced at minimum 20 (max 5%) in both the
contract and CLI.
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 this pull request may close these issues.

2 participants