Sol service/account data conversion - #1596
Merged
Unheilbar merged 6 commits intoJul 30, 2026
Merged
Conversation
product-security-plaid-production
Bot
requested review from
DylanTinianov and
toblich
July 29, 2026 13:23
Contributor
📊 API Diff Results
|
Contributor
📊 API Diff Results
|
Contributor
📊 API Diff Results
|
silaslenihan
approved these changes
Jul 30, 2026
Unheilbar
merged commit Jul 30, 2026
1465951
into
feature/solcap-read_trigger
30 of 39 checks passed
silaslenihan
added a commit
that referenced
this pull request
Aug 3, 2026
* add limits to external requests for solana_service (#1559) * add limits to external requests for solana_service * update tool version * bump golangci lint version * fix linter * update core ref * bump dependnecies * bump dependnecies * fix dependencies * add missing wrpaCtx for rpc calls * implement GetProgramAccounts (#1564) * implement GetProgramAccounts * rollback mocks version * bump common * run tidy * PLEX-2772: Ensure in-memory state stays in sync even on Register failure (#1567) * PLEX-2772: Ensure in-memory state stays in sync even on Register failure * add comments * add comments * PLEX-2770: Sequence number tracking (#1568) * PLEX-2770: Sequence number tracking * make generate * update comment * update filters interface * Throw error on invalid interface * Fix broken test * Fix log poller test * cleanup * Wire balance slot (#1570) * add nil guards for opts * wire Slot for GetFeeForMessage * Skip LogPoller filter errors (#1572) * Skip LogPoller filter errors * fix lint * revert unregister before logic * default to empty struct if opts is nil (#1585) * fix negative from slot for replay command (#1586) * bump to to 1.26.5 (#1587) * fix override passed commitment with default value (#1592) * PLEX-3280/fix cpi filter extractor key collapse (#1591) * Fix cpi event extractor key collapse * Fix cpi event extractor key collapse * add commitment for GetBalance key (#1594) * Optional reply on GetSignatureStatuses (#1595) * Optional reply on GetSignatureStatuses * tidy * fix registration/removal cpi event extractor logic * fix test assertion * Sol service/account data conversion (#1596) * Optional reply on GetSignatureStatuses * tidy * fix registration/removal cpi event extractor logic * fix test assertion * Set only the representation that matches the requested encoding * bump go.mod --------- Co-authored-by: Silas Lenihan <32529249+silaslenihan@users.noreply.github.com> Co-authored-by: Silas Lenihan <sjl@lenihan.net>
Unheilbar
added a commit
that referenced
this pull request
Aug 7, 2026
* add limits to external requests for solana_service (#1559) * add limits to external requests for solana_service * update tool version * bump golangci lint version * fix linter * update core ref * bump dependnecies * bump dependnecies * fix dependencies * add missing wrpaCtx for rpc calls * implement GetProgramAccounts (#1564) * implement GetProgramAccounts * rollback mocks version * bump common * run tidy * PLEX-2772: Ensure in-memory state stays in sync even on Register failure (#1567) * PLEX-2772: Ensure in-memory state stays in sync even on Register failure * add comments * add comments * PLEX-2770: Sequence number tracking (#1568) * PLEX-2770: Sequence number tracking * make generate * update comment * update filters interface * Throw error on invalid interface * Fix broken test * Fix log poller test * cleanup * Wire balance slot (#1570) * add nil guards for opts * wire Slot for GetFeeForMessage * Skip LogPoller filter errors (#1572) * Skip LogPoller filter errors * fix lint * revert unregister before logic * default to empty struct if opts is nil (#1585) * fix negative from slot for replay command (#1586) * bump to to 1.26.5 (#1587) * fix override passed commitment with default value (#1592) * PLEX-3280/fix cpi filter extractor key collapse (#1591) * Fix cpi event extractor key collapse * Fix cpi event extractor key collapse * add commitment for GetBalance key (#1594) * Optional reply on GetSignatureStatuses (#1595) * Optional reply on GetSignatureStatuses * tidy * fix registration/removal cpi event extractor logic * fix test assertion * Sol service/account data conversion (#1596) * Optional reply on GetSignatureStatuses * tidy * fix registration/removal cpi event extractor logic * fix test assertion * Set only the representation that matches the requested encoding * Add TxHash predicate filter e2e test --------- Co-authored-by: Silas Lenihan <32529249+silaslenihan@users.noreply.github.com>
Unheilbar
added a commit
that referenced
this pull request
Aug 10, 2026
* add limits to external requests for solana_service (#1559) * add limits to external requests for solana_service * update tool version * bump golangci lint version * fix linter * update core ref * bump dependnecies * bump dependnecies * fix dependencies * add missing wrpaCtx for rpc calls * implement GetProgramAccounts (#1564) * implement GetProgramAccounts * rollback mocks version * bump common * run tidy * PLEX-2772: Ensure in-memory state stays in sync even on Register failure (#1567) * PLEX-2772: Ensure in-memory state stays in sync even on Register failure * add comments * add comments * PLEX-2770: Sequence number tracking (#1568) * PLEX-2770: Sequence number tracking * make generate * update comment * update filters interface * Throw error on invalid interface * Fix broken test * Fix log poller test * cleanup * Wire balance slot (#1570) * add nil guards for opts * wire Slot for GetFeeForMessage * Skip LogPoller filter errors (#1572) * Skip LogPoller filter errors * fix lint * revert unregister before logic * default to empty struct if opts is nil (#1585) * fix negative from slot for replay command (#1586) * bump to to 1.26.5 (#1587) * fix override passed commitment with default value (#1592) * PLEX-3280/fix cpi filter extractor key collapse (#1591) * Fix cpi event extractor key collapse * Fix cpi event extractor key collapse * add commitment for GetBalance key (#1594) * Optional reply on GetSignatureStatuses (#1595) * Optional reply on GetSignatureStatuses * tidy * fix registration/removal cpi event extractor logic * fix test assertion * Sol service/account data conversion (#1596) * Optional reply on GetSignatureStatuses * tidy * fix registration/removal cpi event extractor logic * fix test assertion * Set only the representation that matches the requested encoding * fix seq num inflation on replay command * make generate * fix linter --------- Co-authored-by: Silas Lenihan <32529249+silaslenihan@users.noreply.github.com>
Unheilbar
added a commit
that referenced
this pull request
Aug 11, 2026
* add limits to external requests for solana_service (#1559) * add limits to external requests for solana_service * update tool version * bump golangci lint version * fix linter * update core ref * bump dependnecies * bump dependnecies * fix dependencies * add missing wrpaCtx for rpc calls * implement GetProgramAccounts (#1564) * implement GetProgramAccounts * rollback mocks version * bump common * run tidy * PLEX-2772: Ensure in-memory state stays in sync even on Register failure (#1567) * PLEX-2772: Ensure in-memory state stays in sync even on Register failure * add comments * add comments * PLEX-2770: Sequence number tracking (#1568) * PLEX-2770: Sequence number tracking * make generate * update comment * update filters interface * Throw error on invalid interface * Fix broken test * Fix log poller test * cleanup * Wire balance slot (#1570) * add nil guards for opts * wire Slot for GetFeeForMessage * Skip LogPoller filter errors (#1572) * Skip LogPoller filter errors * fix lint * revert unregister before logic * default to empty struct if opts is nil (#1585) * fix negative from slot for replay command (#1586) * bump to to 1.26.5 (#1587) * fix override passed commitment with default value (#1592) * PLEX-3280/fix cpi filter extractor key collapse (#1591) * Fix cpi event extractor key collapse * Fix cpi event extractor key collapse * add commitment for GetBalance key (#1594) * Optional reply on GetSignatureStatuses (#1595) * Optional reply on GetSignatureStatuses * tidy * fix registration/removal cpi event extractor logic * fix test assertion * Sol service/account data conversion (#1596) * Optional reply on GetSignatureStatuses * tidy * fix registration/removal cpi event extractor logic * fix test assertion * Set only the representation that matches the requested encoding * implement guard for empty input guard on codec * fix lint --------- Co-authored-by: Silas Lenihan <32529249+silaslenihan@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Requires Dependencies
Resolves Dependencies