Skip to content

Make baseUrl REQUIRED in OAuthClient with clear error messages - #45

Merged
dh7 merged 2 commits into
mainfrom
oauth-dx-improvements
Jan 14, 2026
Merged

dh7 merged 2 commits into
mainfrom
oauth-dx-improvements

Conversation

@dh7

@dh7 dh7 commented Jan 14, 2026

Copy link
Copy Markdown
Owner
  • baseUrl is now a required field (no default)
  • Clear error message with examples if baseUrl not set
  • Validates URL format and warns about wrong domain (mindcache.dev vs api.mindcache.dev)
  • Pings API on init to detect unreachable endpoints
  • Derives authUrl, tokenUrl, userinfoUrl from baseUrl
  • tokenProvider now fetches WebSocket tokens from /api/ws-token
  • Added accessTokenProvider for raw OAuth token access
  • Updated docs with new required baseUrl syntax

- baseUrl is now a required field (no default)
- Clear error message with examples if baseUrl not set
- Validates URL format and warns about wrong domain (mindcache.dev vs api.mindcache.dev)
- Pings API on init to detect unreachable endpoints
- Derives authUrl, tokenUrl, userinfoUrl from baseUrl
- tokenProvider now fetches WebSocket tokens from /api/ws-token
- Added accessTokenProvider for raw OAuth token access
- Updated docs with new required baseUrl syntax
@vercel

vercel Bot commented Jan 14, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
mindcache Ready Ready Preview, Comment Jan 14, 2026 11:06pm

@dh7
dh7 merged commit bdc9c14 into main Jan 14, 2026
9 checks passed

This branch was successfully deployed

1 active deployment
Preview — 275d2bb6 Deployed Jan 14, 2026 by vercel[bot]
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