* 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>

)

* configurable max clock drift

* flag name

* undo spelling fix for consistency

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>

…os#1390)

* Problem: no concrete type registered for type URL of EthAccount

this change should have been included in c0783eb

* Update CHANGELOG.md

…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

Add support for querying the dynamic gas price base fee on Osmosis.

Remove Jack and Mark from the CODEOWNERS file and add Andrew

* Change 2.3.0 to 2.4.0

* Change version in README.MD from 2.4.0 to 2.5.2

---------

Co-authored-by: Justin Tieri <37750742+jtieri@users.noreply.github.com>

…cosmos#1423)

…s#1428)

* base

* typos fixed

* lint links

* relative links

Co-authored-by: Justin Tieri <37750742+jtieri@users.noreply.github.com>

* chore: update dependency versions

* chore: use compatible dependencies so compilation doesn't fail

* chore: use compatible docker version for interchaintest

…osmos#1421)

Bumps [github.com/cosmos/cosmos-sdk](https://github.com/cosmos/cosmos-sdk) from 0.50.4 to 0.50.5.
- [Release notes](https://github.com/cosmos/cosmos-sdk/releases)
- [Changelog](https://github.com/cosmos/cosmos-sdk/blob/v0.50.5/CHANGELOG.md)
- [Commits](cosmos/cosmos-sdk@v0.50.4...v0.50.5)

---
updated-dependencies:
- dependency-name: github.com/cosmos/cosmos-sdk
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

…smos#1422)

Bumps [github.com/cosmos/cosmos-sdk](https://github.com/cosmos/cosmos-sdk) from 0.50.4 to 0.50.5.
- [Release notes](https://github.com/cosmos/cosmos-sdk/releases)
- [Changelog](https://github.com/cosmos/cosmos-sdk/blob/v0.50.5/CHANGELOG.md)
- [Commits](cosmos/cosmos-sdk@v0.50.4...v0.50.5)

---
updated-dependencies:
- dependency-name: github.com/cosmos/cosmos-sdk
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

We currently have commands to query all IBC denom traces or a specific IBC denom trace based off a given hash, but we are lacking the ability to query a hash based off a specific IBC denom trace. This adds a new query to the `Provider` interface that allows users to query this IBC denom trace information for a given chain and hash.

…mos#1437)

* chore: update Penumbra proto generated code and fix import paths

Generate code from the updated Penumbra proto files for v0.70.3 and update the import paths that have changed.

* chore: add missing generated code file

…smos#1440)

Bumps [github.com/cosmos/ibc-go/v8](https://github.com/cosmos/ibc-go) from 8.1.1 to 8.2.0.
- [Release notes](https://github.com/cosmos/ibc-go/releases)
- [Changelog](https://github.com/cosmos/ibc-go/blob/v8.2.0/CHANGELOG.md)
- [Commits](cosmos/ibc-go@v8.1.1...v8.2.0)

---
updated-dependencies:
- dependency-name: github.com/cosmos/ibc-go/v8
  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>

…mos#1441)

Bumps [github.com/cosmos/ibc-go/v8](https://github.com/cosmos/ibc-go) from 8.1.1 to 8.2.0.
- [Release notes](https://github.com/cosmos/ibc-go/releases)
- [Changelog](https://github.com/cosmos/ibc-go/blob/v8.2.0/CHANGELOG.md)
- [Commits](cosmos/ibc-go@v8.1.1...v8.2.0)

---
updated-dependencies:
- dependency-name: github.com/cosmos/ibc-go/v8
  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>

* fix: update query path for getAnchor

* fix: update query path for getAnchor

…onverting to int64 (cosmos#1444)

Previously we were converting a string value to an uint64 which was then cast into an int64 when calling time.Duration. GitHub was throwing a code scanning security alert and this should rectify that issue.