Skip to content

fix(auth): remove Kubernetes service account token fallback#427

Merged
patelspratik merged 1 commit into
mainfrom
BREV-11127/remove-k8s-service-account-token-check
Jul 24, 2026
Merged

fix(auth): remove Kubernetes service account token fallback#427
patelspratik merged 1 commit into
mainfrom
BREV-11127/remove-k8s-service-account-token-check

Conversation

@aditsharma55

@aditsharma55 aditsharma55 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Investigation Summary

The authentication flow checked for a Kubernetes service-account token before reading $HOME/.brev/credentials.json.

Fix

  • Removed the Kubernetes service-account token check from GetAuthTokens.
  • Removed the now-unused GetCurrentWorkspaceServiceToken and getServiceTokenFilePath functions.
  • Brev authentication now reads credentials only from $HOME/.brev/credentials.json.

Relevant Ticket: https://linear.app/nvidia/issue/BREV-11127/brev-cli-incorrectly-prioritizes-kubernetes-serviceaccount-token-over

@aditsharma55
aditsharma55 requested a review from a team as a code owner July 24, 2026 18:49
@patelspratik

Copy link
Copy Markdown
Contributor

can we make sure to update any agent skills and/or documentation that mentions this behavior?

@aditsharma55 aditsharma55 self-assigned this Jul 24, 2026
@aditsharma55

Copy link
Copy Markdown
Contributor Author

can we make sure to update any agent skills and/or documentation that mentions this behavior?

Hi @patelspratik , Thanks for flagging this. I checked all relevant documentation and agent skills for anything that documents or depends on the Kubernetes service-account authentication behavior, but I didn’t find any references to or uses of the removed paths.

Could you please let me know if there’s anything I may have missed?

@patelspratik
patelspratik merged commit 6b111e1 into main Jul 24, 2026
9 checks passed
@patelspratik
patelspratik deleted the BREV-11127/remove-k8s-service-account-token-check branch July 24, 2026 22:03
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