Skip to content
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

🐛 Fix support for strings in OpenAPI status codes: default, 1XX, 2XX, 3XX, 4XX, 5XX #5187

Merged
merged 8 commits into from Oct 14, 2022
Merged

🐛 Fix support for strings in OpenAPI status codes: default, 1XX, 2XX, 3XX, 4XX, 5XX #5187

Changes from 1 commit
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
7e7bdad
allow for 'default' in response in OpenAPI json
JarroVGIT Jul 23, 2022
7b91f25
Expanded the exceptions for is_body_allowed_for_status_code incl test
JarroVGIT Jul 25, 2022
f58f9ba
revert of .gitignore
JarroVGIT Jul 25, 2022
85013ce
🎨 [pre-commit.ci] Auto format from pre-commit.com hooks
pre-commit-ci[bot] Jul 25, 2022
7fe8158
Refactored a little for mypy
JarroVGIT Jul 25, 2022
97e9dcb
refactored
JarroVGIT Jul 25, 2022
a88f50c
♻️ Update acceptable status code string ranges
tiangolo Oct 14, 2022
d66a851
✅ Update test
tiangolo Oct 14, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
2 changes: 1 addition & 1 deletion fastapi/utils.py
View file
Open in desktop
Expand Up @@ -19,7 +19,7 @@


def is_body_allowed_for_status_code(status_code: Union[int, str, None]) -> bool:
if status_code is None:
if status_code is None or status_code == "default":
JarroVGIT marked this conversation as resolved.
Show resolved Hide resolved
return True
current_status_code = int(status_code)
return not (current_status_code < 200 or current_status_code in {204, 304})
Expand Down