Skip to content

feat(chart): optionally sync a Grass proxy template into the proxy secret - #116

Merged
kryptocodes merged 1 commit into
mainfrom
feat/grass-proxy-secret
Oct 28, 2026
Merged

kryptocodes merged 1 commit into
mainfrom
feat/grass-proxy-secret

Conversation

@kryptocodes

Copy link
Copy Markdown
Contributor

Summary

  • Adds secrets.browserRuntimeGrassProxyRemoteKey to the browser-fleet chart (default empty).
  • When set, the browser-runtime proxy ExternalSecret also maps that Secret Manager key to HTTPS_PROXY_URL_GRASS, used by reclaim-api for proxyProvider: "grass" proofs.

Compatibility

Default render is unchanged, so existing ExternalSecrets keep syncing even where the Grass secret does not exist.

Testing

  • helm template with the value unset (unchanged output) and set (extra HTTPS_PROXY_URL_GRASS entry).

…cret

Add secrets.browserRuntimeGrassProxyRemoteKey. When set, the browser-runtime
proxy ExternalSecret also maps that Secret Manager key to HTTPS_PROXY_URL_GRASS,
which reclaim-api uses for proxyProvider=grass proof requests. The value is
empty by default, so existing deployments render and sync exactly as before.

Signed-off-by: kryptocodes <srivatsantb@gmail.com>

@reclaim-pr-agent reclaim-pr-agent Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved by Reclaim PR Agent after an automated GPT-5.6 Terra review.

@reclaim-pr-agent

Copy link
Copy Markdown

Reclaim PR Agent review · run 122780098094

Result Head Blocking findings Ponytail suggestions Check
Approved 51b638e73586 0 0 View run

Reviewed all 1 diff chunks. No blocking defects found.

Ponytail: Lean already. Ship.

@kryptocodes
kryptocodes merged commit 44cc257 into main Oct 28, 2026
18 checks passed
@kryptocodes
kryptocodes deleted the feat/grass-proxy-secret branch October 28, 2026 11:28
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.

2 participants