Skip to content

add field to mark requests as external for solana service#2066

Draft
Unheilbar wants to merge 3 commits into
mainfrom
PLEX-2936/add-external-flag
Draft

add field to mark requests as external for solana service#2066
Unheilbar wants to merge 3 commits into
mainfrom
PLEX-2936/add-external-flag

Conversation

@Unheilbar
Copy link
Copy Markdown
Contributor

@Unheilbar Unheilbar commented May 18, 2026

pkg/http.LimitedTransport moved from chainlink-evm as it can be reused accross multiple chain capabilities.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 18, 2026

✅ API Diff Results - github.com/smartcontractkit/chainlink-common

✅ Compatible Changes (17)

pkg/chains/solana (2)
  • ConvertGetAccountInfoRequestFromProto — ➕ Added

  • ConvertGetAccountInfoRequestToProto — ➕ Added

pkg/chains/solana.(*GetAccountInfoWithOptsRequest) (1)
  • GetIsExternal — ➕ Added
pkg/chains/solana.(*GetMultipleAccountsWithOptsRequest) (1)
  • GetIsExternal — ➕ Added
pkg/chains/solana.(*GetTransactionRequest) (1)
  • GetIsExternal — ➕ Added
pkg/chains/solana.(*SimulateTXRequest) (1)
  • GetIsExternal — ➕ Added
pkg/chains/solana.GetAccountInfoWithOptsRequest (1)
  • IsExternal — ➕ Added
pkg/chains/solana.GetMultipleAccountsWithOptsRequest (1)
  • IsExternal — ➕ Added
pkg/chains/solana.GetTransactionRequest (1)
  • IsExternal — ➕ Added
pkg/chains/solana.SimulateTXRequest (1)
  • IsExternal — ➕ Added
pkg/http (3)
  • GetResponseSizeLimit — ➕ Added

  • LimitedTransport — ➕ Added

  • WithResponseSizeLimit — ➕ Added

pkg/types/chains/solana.GetAccountInfoRequest (1)
  • IsExternal — ➕ Added
pkg/types/chains/solana.GetMultipleAccountsRequest (1)
  • IsExternal — ➕ Added
pkg/types/chains/solana.GetTransactionRequest (1)
  • IsExternal — ➕ Added
pkg/types/chains/solana.SimulateTXRequest (1)
  • IsExternal — ➕ Added

📄 View full apidiff report

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.

1 participant