You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Combo path still gates the 413 context-overflow mapping on stream === true after #4127 #4149
Two gaps in the endpoint trust check applied to the OIDC discovery response, which is the URL that then receives the refresh_token in a POST body:
Suffix-only host match: https://anything.x.ai/token passes. If the discovery document were ever tampered with (or a subdomain compromised), the refresh token would be posted there.
new URL() preserves userinfo, and parsed.toString() returns it: https://u:p@auth.x.ai/oauth2/token passes unchanged, and fetch will turn the userinfo into an Authorization header on the request.
auth.x.ai is the live issuer per https://auth.x.ai/.well-known/openid-configuration (checked 2026-09-09); accounts.x.ai is the second host the progrok reference implementation allow-lists.
Severity: low (discovery is fetched over TLS from a fixed host), filed as hardening. Found while porting this module into ima2-gen (devlog 260909_grok_native_oauth/001_opencodex_port_spec.md D6).
Activity
Merge pull request #4150 from lidge-jun/codex/combo-413-overflow-4149
lidge-jun commentedon Sep 9, 2026
Landed via #4150 at cd813d3
devmerge does not auto-close issues that target the default branch only; this closes the leftover.이 댓글은 grok-bot이 작성했습니다
fix(combos): classify a hopping 413 when the combo exhausts its targe…
fix(combos): classify a hopping 413 when the combo exhausts its targe…
Merge pull request lidge-jun#4150 from lidge-jun/codex/combo-413-over…