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

New workflow #503

Closed
wants to merge 14 commits into from
Closed

New workflow #503

wants to merge 14 commits into from

Conversation

Ciejo
Copy link
Contributor

@Ciejo Ciejo commented Jun 26, 2024

Workflow to build and publish docker images of hyperspace in order to run containers with this image and use relayers.

mina86 and others added 14 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 <[email protected]>
Co-authored-by: Dhruv D Jain <[email protected]>
`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.
@Ciejo Ciejo requested a review from vmarkushin June 26, 2024 17:12
@Ciejo Ciejo closed this Jun 26, 2024
@Ciejo Ciejo deleted the new_workflow branch June 26, 2024 17:55
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.

5 participants