Tags: Eoous/rollkit
Tags
build(deps): Bump golang.org/x/net from 0.26.0 to 0.27.0 (rollkit#1764) Bumps [golang.org/x/net](https://github.com/golang/net) from 0.26.0 to 0.27.0. <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/golang/net/commit/e2310ae9eb6425ee6736cfc40f982f42e20f5850"><code>e2310ae</code></a> go.mod: update golang.org/x dependencies</li> <li><a href="https://github.com/golang/net/commit/77708f716e46255944251965f8240a0eab01e099"><code>77708f7</code></a> quic: skip tests which depend on unimplemented UDP functions on Plan 9</li> <li><a href="https://github.com/golang/net/commit/9617c6335bca5e4e80949a5b1dbe43273260e8a3"><code>9617c63</code></a> http2: avoid Transport hang with Connection: close and AllowHTTP</li> <li>See full diff in <a href="https://github.com/golang/net/compare/v0.26.0...v0.27.0">compare view</a></li> </ul> </details> <br /> [](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) </details> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
fix: Fix vote extension issues, add custom logic to handle secp256k1,… … fix rpc nilptr issues (rollkit#1717) <!-- Please read and fill out this form before submitting your PR. Please make sure you have reviewed our contributors guide before submitting your first PR. NOTE: PR titles should follow semantic commits: https://www.conventionalcommits.org/en/v1.0.0/ --> ## Overview <!-- Please provide an explanation of the PR, including the appropriate context, background, goal, and rationale. If there is an issue with this information, please provide a tl;dr and link the issue. Ex: Closes #<issue number> --> Things fixed: * getAddress does not work for secp256k1, hence using the address directly from the validator set * additional logic to handle secp256k1 signing * move updating store height to before apply block so that, the rpc (like Status) is able to use the updated height instead of old height * change vote extension signing to sign on the bytes from `VoteExtensionSignBytes` instead of plain extension * fix rpc nilptr failures using normalized height * add the missing vote extension enable height from genesis * retain votes from proposed last commit <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Enhanced proposal voting process to support different private key types. - **Refactor** - Updated cryptographic package usage from `ed25519` to `cmcrypto`. - Refactored key handling in multiple functions to support `secp256k1`. - **Tests** - Updated test cases to reflect changes in key handling and cryptographic package usage. <!-- end of auto-generated comment: release notes by coderabbit.ai --> --------- Co-authored-by: Ganesha Upadhyaya <gupadhyaya@Ganeshas-MacBook-Pro-2.local> Co-authored-by: Manav Aggarwal <manavaggarwal1234@gmail.com>
test: Add comprehensive tests for error scenarios in TestVoteExtension ( rollkit#1690) Closes : rollkit#1663 <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Tests** - Introduced new test functions to handle various scenarios related to vote extension: - Preparing proposal with vote extension. - Handling invalid vote extensions. - Testing vote extension failure. - Refactored existing test functions for better modularity and readability. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
docs: Update dependency graph (rollkit#1635) <!-- Please read and fill out this form before submitting your PR. Please make sure you have reviewed our contributors guide before submitting your first PR. NOTE: PR titles should follow semantic commits: https://www.conventionalcommits.org/en/v1.0.0/ --> ## Overview This PR aims to be a starting point for a dependency graph update The update is adding: - more IBC work - connecting Sequencing API and adding Execution - updates to overall progress <!-- Please provide an explanation of the PR, including the appropriate context, background, goal, and rationale. If there is an issue with this information, please provide a tl;dr and link the issue. Ex: Closes #<issue number> --> --------- Co-authored-by: Manav Aggarwal <manavaggarwal1234@gmail.com>
build: update x/net, libp2p and quic (rollkit#1646) <!-- Please read and fill out this form before submitting your PR. Please make sure you have reviewed our contributors guide before submitting your first PR. NOTE: PR titles should follow semantic commits: https://www.conventionalcommits.org/en/v1.0.0/ --> ## Overview This PR updates dependencies and bumps go version used in CI. Replaces rollkit#1617, rollkit#1624. <!-- Please provide an explanation of the PR, including the appropriate context, background, goal, and rationale. If there is an issue with this information, please provide a tl;dr and link the issue. Ex: Closes #<issue number> --> <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Documentation** - Updated the Go version requirement in the README to reflect compatibility changes. - **Chores** - Updated the Go version in the CI workflow to ensure consistent performance and compatibility. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
da: handle tx too large (rollkit#1620) ## Overview This PR updates the block submission loop to additionally handle `tx too large` error. ## Checklist - [x] New and updated code has appropriate documentation - [x] New and updated code has new and/or updated testing - [x] Required CI checks are passing - [x] Visual proof for any user facing features like CLI or documentation updates - [x] Linked issues closed with keywords <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Introduced handling for transactions exceeding the maximum size limit. - **Bug Fixes** - Updated logic to correctly handle oversized transactions. - **Tests** - Added new test cases to verify the handling of transactions that are too large. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
fix(manager): only reset custom gas price (rollkit#1595) ## Overview This PR fixes custom gas price reset after a successful submission. ## Checklist - [x] New and updated code has appropriate documentation - [x] New and updated code has new and/or updated testing - [x] Required CI checks are passing - [x] Visual proof for any user facing features like CLI or documentation updates - [x] Linked issues closed with keywords <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Bug Fixes** - Modified the logic for adjusting `gasPrice` to prevent division operation when not required. - **Refactor** - Refactored test cases for submitting blocks to a mock DA to cover different scenarios. - **Validation** - Added a validation check for `DAGasMultiplier` in `nodeConfig` to meet specific criteria. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
fix(da): increase backoff for mempool errors (rollkit#1535) ## Overview This PR increases the backoff duration for mempool errors, avoiding the scenario where a temporary mempool error e.g due to congestion can exhaust all retries. Fixes rollkit#1522 ## Checklist - [ ] New and updated code has appropriate documentation - [ ] New and updated code has new and/or updated testing - [ ] Required CI checks are passing - [ ] Visual proof for any user facing features like CLI or documentation updates - [ ] Linked issues closed with keywords <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **New Features** - Introduced a default expiration time for mempool transactions to improve transaction management. - Enhanced block submission with handling for `maxBlobSize` and `gasPrice`, offering better flexibility and efficiency. - Added a new flag in `rollkit start` command for setting DA gas price multiplier, facilitating more robust transaction retries. - **Improvements** - Extended the submission timeout period from 60 to 90 seconds, allowing for more reliable block submissions under network delays. - Enriched error messaging and introduced new status codes for block submission to improve debugging and operational transparency. - **Refactor** - Replaced `MockDA` struct with `mock.MockDA` in tests for a more streamlined testing approach. - Implemented a mock `DA` interface in `da/mock/mock.go`, enhancing testability and development efficiency. - **Tests** - Updated various test functions to accommodate new parameters and improved error handling, ensuring the reliability of changes. - **Documentation** - Documented the addition of the DA gas price multiplier flag in `rollkit start` command usage. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
build(deps): bump go-da to v0.2.0 (rollkit#1448) ## Overview This PR updates go-da to v0.2.0 which accepts a context as the first argument to DA interface methods. ## Checklist - [x 4DB9 ] New and updated code has appropriate documentation - [x] New and updated code has new and/or updated testing - [x] Required CI checks are passing - [x] Visual proof for any user facing features like CLI or documentation updates - [x] Linked issues closed with keywords <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit - **Refactor** - Enhanced the data access client to support context parameters for improved operation handling. - **Tests** - Updated tests to align with the new context parameter integration in data access methods. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
PreviousNext