bug: Withdraw liquidity fails — out of gas (400k limit too low) #11
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#11
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
Withdrawing liquidity fails with out of gas error. The hardcoded gas limit (400000) is insufficient — actual usage was 449071.
Error
Transaction failed: out of gas in location: Loading CosmWasm module: execute; gasWanted: 400000, gasUsed: 449071: out of gasExpected
Gas limit for withdraw liquidity should be set higher (e.g. 600000) to accommodate the transaction.
Steps to Reproduce
Fix
In transactions.ts, increase REMOVE_LIQUIDITY_GAS_LIMIT from 400000 to 600000.
Fix committed in
6835c4a:REMOVE_LIQUIDITY_GAS_LIMITincreased from 400k to 600k infrontend-dapp/src/services/terraclassic/transactions.ts. This should resolve the out-of-gas error (actual usage was ~449k). @Brouie please verify when you have a chance.mentioned in commit
6835c4aeaeVerified fixed — REMOVE_LIQUIDITY_GAS_LIMIT bumped to 600000, withdraw works.
mentioned in issue #1
mentioned in issue #18
mentioned in issue #225