Skip to content

fix: handle MiniMax OAuth URL and token expiry#20106

Open
andyst-dev wants to merge 1 commit into
NousResearch:mainfrom
andyst-dev:fix/minimax-oauth-url-expiry
Open

fix: handle MiniMax OAuth URL and token expiry#20106
andyst-dev wants to merge 1 commit into
NousResearch:mainfrom
andyst-dev:fix/minimax-oauth-url-expiry

Conversation

@andyst-dev

@andyst-dev andyst-dev commented May 5, 2026

Copy link
Copy Markdown 
 ## Summary
 - Rewrite MiniMax OAuth authorize URLs from `www.minimax.io` to `platform.minimax.io` when needed.
 - Handle MiniMax `expired_in` values returned either as relative seconds or absolute Unix millisecond timestamps.
 - Add focused tests for URL normalization and token expiry parsing.

 ## Test Plan
 - `python -m pytest tests/test_minimax_oauth.py`

@alt-glitch alt-glitch added type/bug Something isn't working P3 Low — cosmetic, nice to have comp/cli CLI entry point, hermes_cli/, setup wizard area/auth Authentication, OAuth, credential pools provider/minimax MiniMax (Anthropic transport) labels May 5, 2026
@alt-glitch

alt-glitch commented May 5, 2026

Copy link
Copy Markdown
Collaborator

Related to #19461 and #19466 — overlapping MiniMax OAuth URL rewrite fix. This PR additionally handles token expiry parsing.

1 similar comment
@alt-glitch

alt-glitch commented May 5, 2026

Copy link
Copy Markdown
Collaborator

Related to #19461 and #19466 — overlapping MiniMax OAuth URL rewrite fix. This PR additionally handles token expiry parsing.

This was referenced May 5, 2026
Open
Closed
This was referenced May 8, 2026
Closed
Closed
Closed
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

area/auth Authentication, OAuth, credential pools comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have provider/minimax MiniMax (Anthropic transport) type/bug Something isn't working

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@andyst-dev @alt-glitch