Thanks to visit codestin.com
Credit goes to github.com

refactor: unify OAuth providers and support basic auth #18

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

SoldierSacha merged 1 commit into main from codex/analyze-code-review-comments-for-implementation

Aug 15, 2025

Collaborator

SoldierSacha commented Aug 15, 2025

Summary

extract common OAuth discovery and registration into BaseOAuthProvider
handle client_secret_basic and client_secret_post credentials
simplify token exchange and client credentials providers on shared base

Testing

uv run --frozen ruff format .
uv run --frozen ruff check .
uv run --frozen pyright
PYTEST_DISABLE_PLUGIN_AUTOLOAD="" uv run --frozen pytest

https://chatgpt.com/codex/tasks/task_e_689fb3784468832d912938701993b42d


          refactor: unify OAuth providers and support basic auth

bafa7a8

SoldierSacha added the codex label

— with

ChatGPT Codex Connector

SoldierSacha merged commit 10a7a79 into main

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

Labels