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.

Sign up for GitHub

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

Jump to bottom

Solana MessageHasher #15911

Merged
merged 13 commits into from
Jan 30, 2025
Merged

Solana MessageHasher #15911

merged 13 commits into from
Jan 30, 2025

Conversation

prashantkumar1982
Copy link
Contributor

initial implementation

@huangzhen1997 huangzhen1997 changed the base branch from develop to solana-offchain-plugin January 29, 2025 20:21
@huangzhen1997 huangzhen1997 marked this pull request as ready for review January 29, 2025 20:51
@huangzhen1997 huangzhen1997 requested review from a team as code owners January 29, 2025 20:51
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Jan 29, 2025 •
edited
Loading

AER Report: CI Core ran successfully ✅

aer_workflow , commit

AER Report: Operator UI CI ran successfully ✅

aer_workflow , commit

@huangzhen1997 huangzhen1997 requested review from AnieeG, kylesmartin and a team as code owners January 29, 2025 22:26
@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
3 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

@prashantkumar1982 prashantkumar1982 merged commit a81a9b3 into solana-offchain-plugin Jan 30, 2025
202 of 203 checks passed
@prashantkumar1982 prashantkumar1982 deleted the msghasher branch January 30, 2025 22:46
github-merge-queue bot pushed a commit that referenced this pull request Feb 7, 2025
@huangzhen1997 @prashantkumar1982
Solana offchain plugin (#15974)
111fe84 
* cleanup

* update

* fix mod

* update

* revert mod change

* add changeset

* fix smoke test

* fix lint

* revert go mod

* fix

* revert

* revert

* again

* update

* remove file

* revert

* update

* update

* update version

* revert

* downgrade version

* go mod tidy

* tidy

* add token indexes

* update comment

* add solana and remove evm specific

* add solana config

* grammar

* update go mod

* go mod

* update

* update

* changeset

* update

* refactor

* update

* refactor

* update lint

* fix lint

* add chain reader config to plugin creator

* update changeset

* Part 1 for Prashant comments

* refactor getTransmitterKeys

* go mod tidy

* minor

* fix lint

* go import

* gomod

* Blaž comments

* part 1

* fix lint

* data type

* add solana home chain merge func

* rephrase

* add entire config from example

* rm comments

* lint

* lint

* refactor

* update comments

* fix lint

* make generate

* Add solana support to contract transmitter (#15869)

* add solana support to contract transmitter

* add changeset

* one more place

* address unsupported chain family

* encode offramp addr

* fix breaking change

* rename

* test

* revert

* update version

* go mod

* go mod tidy

* go mod

* go.md

* go md

* remove hard coded

* go.md

* go import

* remove unused

* update mod

* update with new generated go binding

* Amit comments

* Solana estimate providers no-op  (#15909)

* no-op for solana estimate providers

* add changeset

* token data noop

* noop for solana rmn

* fix lint

* mod tidy

* Use SVM ABI, needs to check test

* add comment

* goimport

* update

* rename

* go mod tidy

* goimport

* tidy

* refactor

* fix lint

* step 1 for parse extra args map

* add comment

* fix test

* fix

* update

* improvement

* update

* update

* tidy

* update

* fix test

* Matt comments

* address Makram comments

* Makram comments part1

* refactor

* lint

* go mod

* update

* fix

* mod tidy

* go md

* gomd

* fix import

* fix broken integration test

* change commit to little endian, to do for execute

* update

* use LE

* update

* update

* update padding

* fix make

* fix test

* rename

* fix test

* update source chain

* update

* lower case

* update test

* goimport

* minor

* tidy

* implement and add test

* update comments

* goimport

* implement dest exec data handling

* fix conflicts

* fix

* fix

* fix test

* update

* fix test

* update

* add comment

* fix mod

* fix

* refactor

* refactor

* update

* update comment

* add unit test

* update

* lint

* mod

* mod tidy

* fix

* lint

* update

* update

* goimport

* update

* Solana MessageHasher (#15911)

* initial code

* update

* minor

* update test

* update

* lint

* change set

* update

* update

* update token data and test

* update

* fix test

---------

Co-authored-by: Joe Huang <[email protected]>

* mod

* update

* clean up

* update

* update

* add on-chain data decoding unit test coverage

* add comment

* fix import

* more test

* update

* update

* fix

* fix import

* rm unused

* remove unused

* downgrade

* downgrade

* rm

* update

* update

* rm

* update

* revert

* revert

* revert

* fix

* update

* fix

* update

* update

* mod clean

* fix

* mod tidy

* fix

* fix

* update

* fix

* update with develop

* fix

* go mod

* update

* update

* update

* update

* update

* more

* update

* update

* mod

* update

* refactor extradata codec

* update

* goimport

* update

* add error

* tidy

* fix import

* revert previous extra data codec refactor

* fix

---------

Co-authored-by: Prashant Yadav <[email protected]>
karen-stepanyan pushed a commit that referenced this pull request Feb 10, 2025
@huangzhen1997 @prashantkumar1982 @karen-stepanyan
Solana offchain plugin (#15974)
9afe07f 
* cleanup

* update

* fix mod

* update

* revert mod change

* add changeset

* fix smoke test

* fix lint

* revert go mod

* fix

* revert

* revert

* again

* update

* remove file

* revert

* update

* update

* update version

* revert

* downgrade version

* go mod tidy

* tidy

* add token indexes

* update comment

* add solana and remove evm specific

* add solana config

* grammar

* update go mod

* go mod

* update

* update

* changeset

* update

* refactor

* update

* refactor

* update lint

* fix lint

* add chain reader config to plugin creator

* update changeset

* Part 1 for Prashant comments

* refactor getTransmitterKeys

* go mod tidy

* minor

* fix lint

* go import

* gomod

* Blaž comments

* part 1

* fix lint

* data type

* add solana home chain merge func

* rephrase

* add entire config from example

* rm comments

* lint

* lint

* refactor

* update comments

* fix lint

* make generate

* Add solana support to contract transmitter (#15869)

* add solana support to contract transmitter

* add changeset

* one more place

* address unsupported chain family

* encode offramp addr

* fix breaking change

* rename

* test

* revert

* update version

* go mod

* go mod tidy

* go mod

* go.md

* go md

* remove hard coded

* go.md

* go import

* remove unused

* update mod

* update with new generated go binding

* Amit comments

* Solana estimate providers no-op  (#15909)

* no-op for solana estimate providers

* add changeset

* token data noop

* noop for solana rmn

* fix lint

* mod tidy

* Use SVM ABI, needs to check test

* add comment

* goimport

* update

* rename

* go mod tidy

* goimport

* tidy

* refactor

* fix lint

* step 1 for parse extra args map

* add comment

* fix test

* fix

* update

* improvement

* update

* update

* tidy

* update

* fix test

* Matt comments

* address Makram comments

* Makram comments part1

* refactor

* lint

* go mod

* update

* fix

* mod tidy

* go md

* gomd

* fix import

* fix broken integration test

* change commit to little endian, to do for execute

* update

* use LE

* update

* update

* update padding

* fix make

* fix test

* rename

* fix test

* update source chain

* update

* lower case

* update test

* goimport

* minor

* tidy

* implement and add test

* update comments

* goimport

* implement dest exec data handling

* fix conflicts

* fix

* fix

* fix test

* update

* fix test

* update

* add comment

* fix mod

* fix

* refactor

* refactor

* update

* update comment

* add unit test

* update

* lint

* mod

* mod tidy

* fix

* lint

* update

* update

* goimport

* update

* Solana MessageHasher (#15911)

* initial code

* update

* minor

* update test

* update

* lint

* change set

* update

* update

* update token data and test

* update

* fix test

---------

Co-authored-by: Joe Huang <[email protected]>

* mod

* update

* clean up

* update

* update

* add on-chain data decoding unit test coverage

* add comment

* fix import

* more test

* update

* update

* fix

* fix import

* rm unused

* remove unused

* downgrade

* downgrade

* rm

* update

* update

* rm

* update

* revert

* revert

* revert

* fix

* update

* fix

* update

* update

* mod clean

* fix

* mod tidy

* fix

* fix

* update

* fix

* update with develop

* fix

* go mod

* update

* update

* update

* update

* update

* more

* update

* update

* mod

* update

* refactor extradata codec

* update

* goimport

* update

* add error

* tidy

* fix import

* revert previous extra data codec refactor

* fix

---------

Co-authored-by: Prashant Yadav <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@winder winder winder approved these changes

@huangzhen1997 huangzhen1997 huangzhen1997 approved these changes

@cedric-cordenier cedric-cordenier Awaiting requested review from cedric-cordenier cedric-cordenier was automatically assigned from smartcontractkit/foundations

@asoliman92 asoliman92 Awaiting requested review from asoliman92 asoliman92 was automatically assigned from smartcontractkit/ccip-offchain

@makramkd makramkd Awaiting requested review from makramkd makramkd was automatically assigned from smartcontractkit/ccip-offchain

@dimkouv dimkouv Awaiting requested review from dimkouv dimkouv was automatically assigned from smartcontractkit/ccip-offchain

@mateusz-sekara mateusz-sekara Awaiting requested review from mateusz-sekara mateusz-sekara was automatically assigned from smartcontractkit/ccip-offchain

@0xAustinWang 0xAustinWang Awaiting requested review from 0xAustinWang 0xAustinWang was automatically assigned from smartcontractkit/ccip-offchain

@0xnogo 0xnogo Awaiting requested review from 0xnogo 0xnogo was automatically assigned from smartcontractkit/ccip-offchain

@AnieeG AnieeG Awaiting requested review from AnieeG

@kylesmartin kylesmartin Awaiting requested review from kylesmartin

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@prashantkumar1982 @winder @huangzhen1997