Skip to content

fix: normalize MiniMax OAuth expiry and portal URL#22258

Closed
steezkelly wants to merge 1 commit into
NousResearch:mainfrom
steezkelly:fix/minimax-oauth-expiry-url-normalization
Closed

fix: normalize MiniMax OAuth expiry and portal URL#22258
steezkelly wants to merge 1 commit into
NousResearch:mainfrom
steezkelly:fix/minimax-oauth-expiry-url-normalization

Conversation

@steezkelly

Copy link
Copy Markdown
Contributor

Summary

  • Normalize MiniMax OAuth verification URLs from www.minimax.io to the usable platform.minimax.io host while preserving path/query.
  • Normalize MiniMax OAuth expired_in values that arrive as Unix milliseconds into seconds-from-now.
  • Cover both timestamp-shaped and duration-shaped expiry values with focused regression tests.

Test Plan

  • /home/steve/.hermes/hermes-agent/venv/bin/python -m pytest tests/test_minimax_oauth.py -q -o 'addopts='
  • git diff --check

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

Copy link
Copy Markdown
Collaborator

Competing with #20106 and #22240 — all three normalize MiniMax OAuth URL (www→platform.minimax.io) and handle epoch-ms expiry values. Consider consolidating.

@steezkelly

Copy link
Copy Markdown
Contributor Author

Closing this duplicate to reduce maintainer/CI noise. alt-glitch is right that this overlaps the existing MiniMax OAuth PR cluster (#20106/#22240 and related #19461/#19466/#20374/#22020/#21786).\n\nEvidence from comparing diffs:\n- #22258 changes the same files as the other candidates: hermes_cli/auth.py and tests/test_minimax_oauth.py.\n- #20106 already covers both platform.minimax.io URL normalization and epoch-ms expiry parsing with tests.\n- #22240 covers the epoch-ms expiry parsing path on a newer branch.\n\nThis PR added no sufficiently unique value to justify another competing branch. Maintainers should consolidate around one MiniMax OAuth fix rather than review/merge overlapping PRs.

@steezkelly steezkelly closed this May 9, 2026
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.

2 participants