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

feat: solana implementation #13

Draft
wants to merge 108 commits into
base: main
Choose a base branch
from
Draft

feat: solana implementation #13

wants to merge 108 commits into from

Conversation

eloylp
Copy link
Member

@eloylp eloylp commented Apr 1, 2024

Description

This PR groups all the changes that will be upstreamed whenever the ampd Solana implementation is done.

The proposal is to squash this PR when merging.

This will also ensure both, our code and the PR format are OK with the contributing guidelines.

Todos

  • Unit tests
  • Manual tests
  • Documentation
  • Connect epics/issues

Steps to Test

Expected Behaviour

Other Notes

eloylp and others added 30 commits February 6, 2024 20:44
* Add Solana account parsing tools.
* Add solana verification functionality

* Add unit tests to Solana message verifier

* Refactor

* Solana msg verifier reshape (#4)

* remove comment

* Use Solana types from Solana crates

We were using hand crafted Solana types for interacting with RPC.

* refactor: make clippy happy

* Add some traces to Solana msg verifier

* Remove outdated todos

* refactor: re-use already calculated variable

* Clarify test code usage

* Solana msg verifier, try till one program log matches gw event

* Add note to axelar to review when in PR

* fail if tx_id is not found in signatures

* Always fail on chain at handler level

* Handle errors in case a tx cannot be found (finish the handler)

* Remove commented code

* Use original type from solana-axelar repo

* Ensure that we verify sender address in sol msg verifier

* Do not panic on PartialEq

* refactor: clippy

* remove no longer used code

* Add more tests definition to solana msg verify iteration logic

---------

Co-authored-by: eloylp <[email protected]>
Were only intended during development to understand the API.
Co-authored-by: tilacog <[email protected]>
Co-authored-by: tilacog <[email protected]>
Co-authored-by: tilacog <[email protected]>
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