bug: Multi-hop swap out of gas — 600k insufficient for routed swaps #22
Labels
No labels
agent:fix_bugfix
agent:fix_conflicts
agent:fix_security
agent:gap_analysis
agent:implement
agent:implement
agent:implement
agent:open_issues
agent:ready
agent:research
agent:security_audit
agent:verify
architecture
backend
blocker:hybrid
blocker:launch
blocker:limit-orders
blocker:v2
block:log_only
block:security
bug
ci
contracts
correctness
deploy
dev
devops
docs
documentation
duplicate
e2e
enhancement
epic
feature
frontend
functional-completion
gas
good first issue
governance
help wanted
high-risk
hooks
hybrid
indexer
infra
infrastructure
integrators
invalid
launch-blocker
limit-orders
localnet
localterra
low priority
missing-implementation
needs-design
ops
performance
priority
high
priority
medium
product
qa
QA
question
ready
ready
research
scripts
security
security-hardening
smartcontracts
tech-debt
testing
ux
UX
v2
verification
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
code/cl8y-dex-terraclassic#22
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Description
Multi-hop swaps via the router fail with out of gas. The SWAP_GAS_LIMIT (600000) is sufficient for single-pair swaps but not for multi-hop routes.
Error
out of gas in location: wasm contract; gasWanted: 600000, gasUsed: 601616: out of gasRoute Tested
EMBER → JADE → TOPAZ (2-hop route)
Expected
Gas limit should scale based on number of hops. Suggest 600k per hop (e.g. 2 hops = 1200k).
Steps to Reproduce
Notes
Single-pair swaps work fine at 600k. This is a new issue since the router was just implemented.
mentioned in issue #18
mentioned in commit
217d253ad3mentioned in merge request !725
mentioned in commit
7790a6d69bmentioned in commit
46a71f04dd@Brouie Please verify fix.
mentioned in commit
506165730dmentioned in merge request !727
mentioned in merge request !728
mentioned in commit
11bd1887f0mentioned in commit
911c542e68mentioned in commit
be5544adfd