Skip to content

fix(auth): rewrite stale MiniMax OAuth approval URL#19461

Open
LeonSGP43 wants to merge 1 commit into
NousResearch:mainfrom
LeonSGP43:codex/minimax-oauth-verification-uri
Open

fix(auth): rewrite stale MiniMax OAuth approval URL#19461
LeonSGP43 wants to merge 1 commit into
NousResearch:mainfrom
LeonSGP43:codex/minimax-oauth-verification-uri

Conversation

@LeonSGP43

@LeonSGP43 LeonSGP43 commented May 4, 2026

Copy link
Copy Markdown
Contributor

Summary

  • rewrite stale MiniMax OAuth approval URLs from www.minimax.io/oauth-authorize to platform.minimax.io/oauth-authorize
  • preserve the original path, query parameters, and unrelated verification URLs
  • cover the rewrite and preservation behavior in the MiniMax OAuth tests

Why

Fixes #19337. The current MiniMax OAuth code trusts the portal response verification URL directly. Live checks show https://www.minimax.io/oauth-authorize?user_code=TEST&client=OpenClaw redirects away to the homepage with 307, while https://platform.minimax.io/oauth-authorize?user_code=TEST&client=OpenClaw returns 200.

Test

  • scripts/run_tests.sh tests/test_minimax_oauth.py

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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

Development

Successfully merging this pull request may close these issues.

MiniMax OAuth returns stale verification_uri pointing to deleted /oauth-authorize page on www.minimax.io

2 participants