feat(proxy): add account-bound Codex upstream routing#873
Conversation
github-actions
Bot
added
the
db migration
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 38b24e0132
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Komzpa
force-pushed
the
repair/pr-869-proxy-pool-egress
branch
from
38b24e0 to
1aaef2c
Compare
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 1aaef2cf20
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 471cf0439b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@codex review |
|
Codex Review: Didn't find any major issues. Nice work! ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
# Conflicts: # app/core/auth/dependencies.py # app/core/auth/refresh.py # app/core/clients/files.py # app/core/clients/model_fetcher.py # app/core/clients/oauth.py # app/core/clients/proxy.py # app/core/clients/proxy_websocket.py # app/core/clients/usage.py # app/core/openai/model_refresh_scheduler.py # app/modules/accounts/auth_manager.py # app/modules/accounts/repository.py # app/modules/oauth/service.py # app/modules/proxy/service.py # app/modules/usage/updater.py # tests/integration/test_proxy_files.py # tests/unit/test_auth_manager.py # tests/unit/test_proxy_websocket_client.py
3 participants
Supersedes #869 after the original pull ref stopped following the repaired source branch.
This carries the account-bound upstream proxy routing branch with the latest routed OAuth/bootstrap, settings validation, transcribe transport, and route failure metadata fixes.
Validation run locally on the latest repair commit:
uv run pytest tests/integration/test_oauth_flow.py -quv run ruff check app/modules/oauth/service.py tests/integration/test_oauth_flow.pyuv run ty check app/modules/oauth/service.py tests/integration/test_oauth_flow.pyOriginal PR: #869.