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

Merge master into main #512

Closed
wants to merge 15 commits into from
Closed

Merge master into main #512

wants to merge 15 commits into from

Conversation

Sventimir
Copy link
Contributor

Pull latest changes (in particular making GRPC and WS urls optional in hyperspace) from master to main.

mina86 and others added 15 commits December 13, 2023 10:04
Introduce From and TryFrom conversion between IBC identifier types in
our fork of ibc-rs and upstream ibc-rs crate.  This will be needed by
Solana implementation which uses upstream code.
* Fix overflow

* Revert "subxt update (#462)"

This reverts commit 687ff2f.
* adding new things to relayer documentation

* adding connection toml

* removing spaces

* removing ips

* modifying contracts
* Fix cosmos client next updates and improve some of the queries

* Return in `MetricsHandler::handle_events` if no new height was found
…#478)



---------

Signed-off-by: Michal Nazarewicz <mina86@mina86.com>
Co-authored-by: Dhruv D Jain <dhruv@iamsizzling.com>
`verify_with_voter_set` function is performing additional validation that go beyond the [standard implementation](https://raw.githubusercontent.com/w3f/consensus/master/pdf/grandpa.pdf) (section 2.3), possibly leading to proofs not being accepted while valid.

Instead, we need to accept all finality justifications where is_valid is true. The equivocations or other misbehaviors detected in the justifications must be handled too but not should not reject the proof.
Make GRPC and WS urls for cosmos optional

Co-authored-by: Vladislav Markushin <negigic@gmail.com>
@Sventimir Sventimir self-assigned this Aug 13, 2024
@Sventimir Sventimir closed this Aug 13, 2024
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.

6 participants