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

Skip to content

🐛 Check Content-Type request header before assuming JSON #2118

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

Merged
merged 12 commits into from
Jun 7, 2021
Merged

🐛 Check Content-Type request header before assuming JSON #2118

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
e46bce3
Check Content-Type before assuming JSON
Oct 2, 2020
df4fdf0
Remove now-unneccessary exception handling
Oct 2, 2020
4e8605e
Fix broken-body test
Oct 2, 2020
c7191f5
Assume "Content-Type": "application/octet-stream"
Oct 2, 2020
c169258
Explicitly specify Content-Type in gzip test
Oct 2, 2020
d644c30
Interpret as JSON any mimetype ending with "json"
Oct 2, 2020
6ecfba9
Generalize media type parsing to handle parameters
Oct 3, 2020
d752efd
Add test for handling various Content-Type headers
patrickkwang Jan 20, 2021
f80305d
✅ Update tests for content-type headers and include extra corner case
tiangolo Jun 7, 2021
344ae3f
♻️ Re-implement JSON content-type parsing with Python's standard libr…
tiangolo Jun 7, 2021
29257d0
🔀 Merge master to update codebase, dependencies and tests
tiangolo Jun 7, 2021
5729e8a
✅ Include again test with patch to force exception, for coverage
tiangolo Jun 7, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Remove now-unneccessary exception handling
  • Loading branch information
@patrickkwang
Patrick Wang authored and patrickkwang committed Apr 10, 2021
commit df4fdf0c7e092550d1e5b1633f10f41d713bbd66
3 changes: 0 additions & 3 deletions fastapi/routing.py
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,6 @@
import asyncio
import enum
import inspect
import json
from typing import (
Any,
Callable,
Expand Down Expand Up @@ -188,8 +187,6 @@ async def app(request: Request) -> Response:
body = await request.json()
else:
body = body_bytes
except json.JSONDecodeError as e:
raise RequestValidationError([ErrorWrapper(e, ("body", e.pos))], body=e.doc)
except Exception as e:
raise HTTPException(
status_code=400, detail="There was an error parsing the body"
Expand Down