Skip to content

Handle raw token response from Fly OIDC endpoint#11659

Merged
nbudin merged 1 commit into
mainfrom
fix-oidc-raw-token
Jun 9, 2026
Merged

Handle raw token response from Fly OIDC endpoint#11659
nbudin merged 1 commit into
mainfrom
fix-oidc-raw-token

Conversation

@nbudin

@nbudin nbudin commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Purpose

After #11658 fixed the socket permission issue (running as root), the next failure showed the endpoint returns the JWT as a bare string rather than {"token":"..."} JSON. The Python one-liner was expecting a JSON object and exited non-zero.

Now tries JSON-parse first; if that fails (or there's no token key), uses the response directly as the token value.

🤖 Generated with Claude Code

The endpoint returns the JWT token as a bare string, not wrapped in
{"token":"..."} JSON. python3 was failing to parse it as JSON and
exiting non-zero. Now try JSON first; fall back to treating the whole
response as the token value.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@nbudin nbudin marked this pull request as ready for review June 9, 2026 04:27
@nbudin nbudin merged commit 1b776b1 into main Jun 9, 2026
12 checks passed
@nbudin nbudin deleted the fix-oidc-raw-token branch June 9, 2026 04:27
@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Code Coverage Report: Only Changed Files listed

Package Base Coverage New Coverage Difference
Overall Coverage 🟢 52.92% 🟢 52.92% ⚪ 0%

Minimum allowed coverage is 0%, this run produced 52.92%

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