Skip to content

feat(balance): add --balance.token for authenticated gRPC endpoints - #62

Merged
auricom merged 1 commit into
mainfrom
recur/cozy-potpie
Nov 6, 2026
Merged

auricom merged 1 commit into
mainfrom
recur/cozy-potpie

Conversation

@auricom

@auricom auricom commented Nov 6, 2026

Copy link
Copy Markdown
Collaborator

Adds a --balance.token flag. Its value is sent as x-token gRPC metadata on every balance query to every URL in --balance.consensus-rpc-urls, which DA-proxy gRPC endpoints require. It is kept separate from --celestia-token so the DA token isn't sent to third-party consensus nodes. The token is also sent over plaintext http:// endpoints; the code notes switching to per-RPC credentials with TLS enforcement as the upgrade path. cosmos.NewClient and balance.NewMetricsExporter take the new token argument. A new test checks the header is sent only when a token is set, and the README documents the flag.

🤖 Generated with Claude Code

Add a --balance.token flag whose value is sent as x-token gRPC metadata
on every balance query to each consensus RPC URL, as required by
DA-proxy endpoints. It is separate from --celestia-token so the DA token
is not sent to third-party consensus nodes.

The token is threaded from the flag through balance.NewMetricsExporter
into cosmos.NewClient. Tests cover the header being sent and omitted,
and the README documents the new flag.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Nov 6, 2026

Copy link
Copy Markdown

Important

  • Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 9b3ada06-bd57-4c20-b72d-9e1dd754a20a
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Tip

🎯 Customize your risk assessments

Guide how CodeRabbit assesses merge risk and security architecture risk. Highlight sensitive changes and explain your security boundaries using Merge Risk Instructions and Security Risk Instructions in your review settings. Security architecture risk assessments require the Advanced plan or above.

✅ Automatic approvals, with your rules

Let CodeRabbit approve pull requests once required threads are resolved, pre-merge checks pass, and your risk limits are met. Enable Automatic approvals in review settings, with overrides for teams or file paths.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@auricom
auricom merged commit 32aa0bd into main Nov 6, 2026
5 checks passed
@auricom
auricom deleted the recur/cozy-potpie branch November 6, 2026 10:26
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