Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(gomod): rollback cometbft to v0.38.9 #406

Merged
merged 3 commits into from
Dec 6, 2024
Merged

Conversation

0xHansLee
Copy link
Contributor

After upgrading cometbft to v0.38.12, we encountered sent next PEX request too soon error.
Before fixing this issue, I would like to rollback the cometbft to v0.38.9 (go-ethereum and btcec as well since there are related breaking changes from v0.38.9 to v0.38.12).

issue: none

@@ -23,7 +23,7 @@ require (
github.com/cosmos/cosmos-sdk v0.50.7
github.com/cosmos/gogoproto v1.7.0
github.com/decred/dcrd/dcrec/secp256k1/v4 v4.3.0
github.com/ethereum/go-ethereum v1.14.8
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we also need to rollback geth version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, because of the compatibility issue of btcec.
cometbft v0.38.9 uses v2.2.0 of btcec, but go-ethereum v1.14.8 uses v2.3.4 of btcec, which is not backward compatible.

@ezreal1997
Copy link
Contributor

do we also need to rollback cosmos-sdk to v0.50.8? I remember the reason to upgrade cosmos-sdk to v0.50.10 is to be compatible with cometBFT v0.38.12

@0xHansLee
Copy link
Contributor Author

do we also need to rollback cosmos-sdk to v0.50.8? I remember the reason to upgrade cosmos-sdk to v0.50.10 is to be compatible with cometBFT v0.38.12

It was (but not required). But it seems v0.50.10 is also compatible with cometbft v0.38.9 (57block tested). So I just revert cometbft since there are some changes you made in cosmos-sdk as well.

@edisonz0718 edisonz0718 force-pushed the hans/rollback-cometbft branch from b1fdfaa to 172c5ff Compare December 6, 2024 17:53
@edisonz0718 edisonz0718 merged commit 450dcce into main Dec 6, 2024
8 checks passed
@edisonz0718 edisonz0718 deleted the hans/rollback-cometbft branch December 6, 2024 18:19
Copy link

github-actions bot commented Dec 6, 2024

Binary uploaded successfully 🎉

📦 Version Name: 0.13.2-unstable-450dcce
📦 Download Source: AWS S3

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.

4 participants