forked from wormhole-foundation/wormhole
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Showing
2 changed files
with
21 additions
and
0 deletions.
There are no files selected for viewing
This file contains 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
19 changes: 19 additions & 0 deletions
19
wormchain/docs/architecture/0006-transfer-wasmd-fork-logic-to-wormchain.md
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,19 @@ | ||
# 6. Transfer Wasmd Fork Logic to Wormchain Ante Handler | ||
|
||
Date: 2024-07-29 | ||
|
||
## Status | ||
|
||
Accepted | ||
|
||
## Context | ||
|
||
The Wormchain repo depends on a fork of `wasmd` to prevent certain types of Wasm transactions from being accepted by the chain. The Strangelove team proposed migrating the logic to an Ante Handler on chain to remove the need to maintain a fork of `wasmd`. | ||
|
||
## Decision | ||
|
||
After some discussion and investigation, the Strangelove team has decided to transfer the fork logic to an Ante Handler on Wormchain. This will allow the Wormchain to be more in line with the Cosmos SDK and reduce the maintenance overhead of maintaining a fork of `wasmd`. | ||
|
||
## Consequences | ||
|
||
The Wormchain will no longer depend on a fork of `wasmd` and will be able to upgrade to newer versions of the Cosmos SDK more easily. The Strangelove team will need to implement the Ante Handler and test it to ensure that it works as expected. As of now, the Strangelove team has not identified any major risks associated with this decision. |