Skip to content

coinHelpers: grpcToken param for authenticated gRPC endpoints - #174

Closed
11felix wants to merge 2 commits into
mainfrom
feature/coin-helpers-grpc-token
Closed

coinHelpers: grpcToken param for authenticated gRPC endpoints#174
11felix wants to merge 2 commits into
mainfrom
feature/coin-helpers-grpc-token

Conversation

@11felix

@11felix 11felix commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Adds an optional grpcToken to getCoinObjectCounts / buildMergeCoinsTransaction (threaded to the internal client as x-token metadata), so alphafi-admin's merge flow can authenticate against BlockPI when it passes a keyed endpoint. No behavior change when omitted. Companion to alphafi-sdk-js#99; consumed by alphafi-admin#34.

@11felix

11felix commented Aug 14, 2026

Copy link
Copy Markdown
Contributor Author

ff86d15 adopts the fix from the meta-dropping finding on #176/#99 (Zorag44): grpcClient now builds the GrpcWebFetchTransport explicitly — SuiGrpcClient's convenience constructor silently drops meta, so the previous shape never sent the token. @protobuf-ts/grpcweb-transport added as a direct dependency. Verified live against BlockPI. #176 stacks on this branch so the dependency is shared.

@11felix

11felix commented Aug 14, 2026

Copy link
Copy Markdown
Contributor Author

Consolidated into #176 per khush — same body of work; this branch's commits (5a207b0, ff86d15) are included there, so nothing is lost. #176 now targets main with the combined diff.

@11felix 11felix closed this Aug 14, 2026
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