chore: rebase to cosmos/relayer/2.5.2 #14
Merged
Conversation
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
* fix typos in comments * docs: fix typos
* client state unbonding period * remove else
* fix: add os.IsExist check for err * Update CHANGELOG.md * Apply suggestions from code review
* feat: add log-level in global config * allow config logger get overwritten * register log-level flag * allow flag overwrite config * allow debug flag overwrite log level * new logger when no config file provided * reuse with initLogger
* fix: cleanup close init after close confirmed * Update CHANGELOG.md
…dle breaking changes (cosmos#1312) * upgrade to deps for ibc-go v8 and sdk v0.50.0 * delete old penumbra protos * move cometbft block results helper methods and types to their own go file * remove commented out code * add go.work file back * update keys restore test with proper expected address * update penumbra protos to v0.62.0 * ci: add setup Go step to prepare-scenario-matrix * ci: filter out duplicate proto warnings when generating test matrix * deps: bump ibc-go and cosmos sdk to stable releases
* unbonding period info * remove extra fmt.sprint * mod tidy
* deps: bump version numbers for handful of deps * chore: go mod tidy * test: update e2e test cases to account for recent changes in interchaintest
* strangelove project managment * file rename
* fix: cosmos_relayer_tx_failure metric * feedback
* Ensure that ✘ is displayed for clients if they are expired * Address review comment
* penumbra: update proof spec to new multistore format * penumbra: use new commitment prefix --------- Co-authored-by: Justin Tieri <37750742+jtieri@users.noreply.github.com>
…osmos#1358) Co-authored-by: Justin Tieri <37750742+jtieri@users.noreply.github.com>
Co-authored-by: Justin Tieri <37750742+jtieri@users.noreply.github.com>
…t upgrade (cosmos#1360) * chore: update penumbra protos * fix: get event parsing working after cometbft upgrade * chore: update penumbra protos
…MsgRegisterCounterpartyPayee` (cosmos#1362) * chore: fix linter errors * fix: pipe memo from cli to transfer and RegisterCounterpartyPayee msgs + fix linter errors
* fix: restructure `ShouldRelayChannel` * fix test * check channel open + tests * remove open channel check * fix race - remove warn log * add mutex's * Update comment
* Problem: simulate gas is less than require with feegrants * Update CHANGELOG.md
* troubleshoot doc improvment fix md fix md fix edit lines fix lines fix lines capitalize * feedback
…dgments` (cosmos#1356) * unrelayed packets metric * feedback
* adding tp percentage flag * fix duration flub * nit fixes
Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.15.0 to 0.17.0. - [Commits](golang/crypto@v0.15.0...v0.17.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
) Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.15.0 to 0.17.0. - [Commits](golang/crypto@v0.15.0...v0.17.0) --- updated-dependencies: - dependency-name: golang.org/x/crypto dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…cosmos#1370) Bumps [github.com/dvsekhvalnov/jose2go](https://github.com/dvsekhvalnov/jose2go) from 1.5.0 to 1.6.0. - [Commits](dvsekhvalnov/jose2go@v1.5...v1.6.0) --- updated-dependencies: - dependency-name: github.com/dvsekhvalnov/jose2go dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…osmos#1371) Bumps [github.com/dvsekhvalnov/jose2go](https://github.com/dvsekhvalnov/jose2go) from 1.5.0 to 1.6.0. - [Commits](dvsekhvalnov/jose2go@v1.5...v1.6.0) --- updated-dependencies: - dependency-name: github.com/dvsekhvalnov/jose2go dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…cosmos#1372) Bumps [github.com/containerd/containerd](https://github.com/containerd/containerd) from 1.7.7 to 1.7.11. - [Release notes](https://github.com/containerd/containerd/releases) - [Changelog](https://github.com/containerd/containerd/blob/main/RELEASES.md) - [Commits](containerd/containerd@v1.7.7...v1.7.11) --- updated-dependencies: - dependency-name: github.com/containerd/containerd dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* add ICS-29 guide to path docs * Update create-path-across-chain.md * change hyperlink --------- Co-authored-by: Justin Tieri <37750742+jtieri@users.noreply.github.com>
This update introduces a new case statement for *chantypes.MsgTimeout within the Penumbra chain logic. This will correctly construct and return an Action_IbcRelayAction when a MsgTimeout is encountered.
* Create SECURITY.md * Update README.md with security instructions
…mos#1384) * feat: introduce cometbft client wrapper to eliminate forked repo * feat: finish implementing the CometRPC interface * chore: clean up * chore: use relayer client wrapper to get upstream comet types
…smos#1377) * Add ics20 memo limit * Add max receiver size and ics-20 memo limit configuration This change introduces new configuration, MaxReceiverSize & ICS20MemoLimit, in the relayer, path processor, and the global config. It updates the existing functions and methods to utilize this new configuration property for managing and validating receiver and memo sizes in IBC related tasks. * Add WriteConfig function and update method names Added a new function WriteConfig in relayertest/system.go for testing. Also, updated method names in relayer.go to start with an uppercase letter for public visibility. Besides, upgraded Docker image version in localhost_client_test.go from v8.0.0-beta.1 to v8.0.0. * test: add memo and receiver limit test Created the Memo and Receiver Limit test within the interchain test suite. This test evaluates the functionality of IBC transfers with a memo field length exceeding configured limit, as well as receiver field exceeding the limit. It also evaluates a successful IBC transfer for completeness. * test: update test for memo and receiver limit Adjusted the test case 'TestMemoAndReceiverLimit' to focus more on the functionality of sending transfers with a memo and receiver limit configured. This reduces the complexity of the test case, and it now focuses more closely on the actual purpose of the method. The general flow of the test has been maintained, but some unnecessary operations have been removed. * fix: properly unmarshal ics-20 packet data & remove debug print statements This commit removes various debug print statements from the code, making it cleaner and easier to read. Additionally, it updates the way the 'transfertypes.ModuleCdc.Unmarshal' function is used to 'transfertypes.ModuleCdc.UnmarshalJSON', simplifying the overall structure of the code. * test: expand test users and IBC transfers in memo/receiver limit test The test for the memo receiver limit now supports four test users instead of two, allowing for more complex transfer scenarios. This change requires increased checks for account balances, as well as additional IBC transfers. This expansion helps ensure more comprehensive testing coverage and overall system validation. --------- Co-authored-by: Andrew Gouin <andrew@gouin.io>
* Allow feegrants where granter is external (relayer does not have key) * linting/logging * Added feegrant command gas flag * Added doc for external feegrant config * Fix logging to conform to style guide * Lock on read bech32 address map * Lock on read bech32 address map * Lock on read bech32 address map --------- Co-authored-by: Kyle <kyle@strange.love>
* refactor: encapsulate slim client in wrapper * test: fix missed change when handling merge conflicts
* Remove writeack too when removing packet retention * Add additional state tracking to prevent loops * simplify * fix
…1379) * update penumbra protos and fix tx.go, query.go with new apis * remove deprecated narsil * add missing protos --------- Co-authored-by: Justin Tieri <37750742+jtieri@users.noreply.github.com>
…osmos#1392) Bumps [github.com/opencontainers/runc](https://github.com/opencontainers/runc) from 1.1.5 to 1.1.12. - [Release notes](https://github.com/opencontainers/runc/releases) - [Changelog](https://github.com/opencontainers/runc/blob/v1.1.12/CHANGELOG.md) - [Commits](opencontainers/runc@v1.1.5...v1.1.12) --- updated-dependencies: - dependency-name: github.com/opencontainers/runc dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* retry only if processing completed for a packet * nil safety * keep assembled and lastProcessedHeight * use counterparty key for remove packet retention * Use channel for finishedProcessing
Bumps [github.com/moby/moby](https://github.com/moby/moby) from 24.0.7+incompatible to 24.0.9+incompatible. - [Release notes](https://github.com/moby/moby/releases) - [Commits](moby/moby@v24.0.7...v24.0.9) --- updated-dependencies: - dependency-name: github.com/moby/moby dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* fix: use context timeout per paginated query Previously we were using one 60s timeout for the entire paginated query request for channels. This change moves the context timeout to be on a per-request basis and uses a 10s timeout per request. * refactor: break out anonymous function into a named method to use in both channel query methods * refactor: avoid redundant code and use QueryChannelsPaginated in QueryChannels
… constructed (cosmos#1407) * fix: only send client updates when necessary and when msg is properly constructed Previously in the relayer we were attempting to send a `MsgUpdateClient` when `needsClientUpdate` was true, as determined in the `shouldUpdateClientNow` method, without considering if the message was properly constructed in `assembleMsgUpdateClient`. This adds a check in `trackAndSendMessages` to ensure we only attempt to send the msg when a client update is necessary AND the `MsgUpdateClient` was properly constructed. * fix: undo previous check in `sendBatchMessages` * chore: update cometbft-client to v0.1.0 * chore: update cometbft-client to v0.1.0 in e2e tests
…in halt in Go cosmos cometbft (cosmos#1398) A vulnerability in CometBFT’s validation logic for `VoteExtensionsEnableHeight` can result in a chain halt when triggered through a governance parameter change proposal on an ABCI2 Application Chain. If a parameter change proposal including a `VoteExtensionsEnableHeight` modification is passed, nodes running the affected versions may panic, halting the network. Co-authored-by: Justin Tieri <37750742+jtieri@users.noreply.github.com>
…smos#1386) Bumps [github.com/cometbft/cometbft](https://github.com/cometbft/cometbft) from 0.38.2 to 0.38.5. - [Release notes](https://github.com/cometbft/cometbft/releases) - [Changelog](https://github.com/cometbft/cometbft/blob/v0.38.5/CHANGELOG.md) - [Commits](cometbft/cometbft@v0.38.2...v0.38.5) --- updated-dependencies: - dependency-name: github.com/cometbft/cometbft dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Justin Tieri <37750742+jtieri@users.noreply.github.com>
* deps: update deps to latest patch versions * tests: update chain binaries to more recent tags, clean up code
omritoptix
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.