Skip to content

feat(go): detect RAPT auth errors and auto-detect credential type#141

Merged
lidavidm merged 1 commit intoadbc-drivers:mainfrom
spiceai:lukim/improve-auth-error-handling
Mar 24, 2026
Merged

feat(go): detect RAPT auth errors and auto-detect credential type#141
lidavidm merged 1 commit intoadbc-drivers:mainfrom
spiceai:lukim/improve-auth-error-handling

Conversation

@lukekim
Copy link
Copy Markdown
Contributor

@lukekim lukekim commented Mar 24, 2026

What's Changed

  • Parse OAuth error responses in getAccessToken() instead of silently swallowing them
  • Detect RAPT (re-authentication policy) errors and surface actionable guidance to use service accounts or re-authenticate via gcloud
  • Extract shared isReauthError() helper and reauthGuidance constant to avoid duplication between connection.go and util.go
  • Auto-detect credential type from JSON file/string when not explicitly set, instead of passing empty/Unknown type to the Google SDK
  • Add debug-level logging for which auth path is active

…al type, add debug logging

- Parse OAuth error responses in getAccessToken() instead of silently
  swallowing them
- Detect RAPT (re-authentication policy) errors and surface actionable
  guidance to use service accounts or re-authenticate via gcloud
- Extract shared isReauthError() helper and reauthGuidance constant to
  avoid duplication between connection.go and util.go
- Auto-detect credential type from JSON file/string when not explicitly
  set, instead of passing empty/Unknown type to the Google SDK
- Add debug-level logging for which auth path is active
@lidavidm lidavidm changed the title Improve auth error handling: detect RAPT errors, auto-detect credential type, add debug logging feat(go): improve auth error handling: detect RAPT errors, auto-detect credential type, add debug logging Mar 24, 2026
@lidavidm lidavidm changed the title feat(go): improve auth error handling: detect RAPT errors, auto-detect credential type, add debug logging feat(go): detect RAPT auth errors and auto-detect credential type Mar 24, 2026
@github-actions
Copy link
Copy Markdown

@lidavidm
Copy link
Copy Markdown
Contributor

Hmm, flaky test, wonder why BigQuery's API is so weird...

@lidavidm lidavidm merged commit 6941fd1 into adbc-drivers:main Mar 24, 2026
3 of 9 checks passed
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