Skip to content

fix: probe Codex auth with the account's default model - #570

Open
hacktivistxt wants to merge 1 commit into
GreyDGL:mainfrom
hacktivistxt:fix/codex-auth-probe-default-model
Open

hacktivistxt wants to merge 1 commit into
GreyDGL:mainfrom
hacktivistxt:fix/codex-auth-probe-default-model

Conversation

@hacktivistxt

Copy link
Copy Markdown

The live Codex auth checks in scripts/docker-auth-status.sh and scripts/docker-login.sh probed with -m gpt-5.4-mini, which a ChatGPT-account login rejects ("not supported when using Codex with a ChatGPT account"). A valid login was therefore reported as codex: NOT logged in, and docker-login.sh would force a needless re-login.

This drops the explicit -m flag so the probe uses the account's default model (verified working against a ChatGPT-account login).

🤖 Generated with Claude Code

The live Codex checks in docker-auth-status.sh and docker-login.sh passed
`-m gpt-5.4-mini`, which a ChatGPT-account login rejects ("not supported
when using Codex with a ChatGPT account"). A valid login was therefore
reported as NOT logged in, and docker-login.sh would force a needless
re-login. Drop the explicit model so the probe uses the account default.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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