Skip to content

✏️ Fix typo for client_secret in OAuth2 form docstrings#14946

Merged
svlandeg merged 1 commit intofastapi:masterfrom
bysiber:fix/client-password-typo-oauth2
Mar 24, 2026
Merged

✏️ Fix typo for client_secret in OAuth2 form docstrings#14946
svlandeg merged 1 commit intofastapi:masterfrom
bysiber:fix/client-password-typo-oauth2

Conversation

@bysiber
Copy link
Copy Markdown
Contributor

@bysiber bysiber commented Feb 20, 2026

The client_secret parameter docstring in both OAuth2PasswordRequestForm and OAuth2PasswordRequestFormStrict incorrectly referred to client_password instead of client_secret.

The parameter itself is correctly named client_secret, and the rest of each docstring also uses client_secret — it's just this one mention that had the wrong name.

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Feb 20, 2026

Merging this PR will not alter performance

✅ 20 untouched benchmarks


Comparing bysiber:fix/client-password-typo-oauth2 (2c09b63) with master (faee822)1

Open in CodSpeed

Footnotes

  1. No successful run was found on master (c441583) during the generation of this report, so faee822 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

@bysiber bysiber force-pushed the fix/client-password-typo-oauth2 branch from f562377 to 2c09b63 Compare February 20, 2026 00:34
@YuriiMotov YuriiMotov added the docs Documentation about how to use FastAPI label Feb 20, 2026
Copy link
Copy Markdown
Member

@YuriiMotov YuriiMotov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Copy link
Copy Markdown
Member

@svlandeg svlandeg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @bysiber! 🎉

@svlandeg svlandeg changed the title ✏️ Fix client_password typo in OAuth2 form docstrings ✏️ Fix typo for client_secret in OAuth2 form docstrings Mar 24, 2026
@svlandeg svlandeg merged commit fa3588c into fastapi:master Mar 24, 2026
30 of 31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation about how to use FastAPI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants