Skip to content

Add exception handler to unsupported media type case#4615

Closed
pedroven wants to merge 2 commits intofastapi:masterfrom
pedroven:master
Closed

Add exception handler to unsupported media type case#4615
pedroven wants to merge 2 commits intofastapi:masterfrom
pedroven:master

Conversation

@pedroven
Copy link
Copy Markdown

Resolve #4550

Comment thread fastapi/routing.py Outdated
Comment thread fastapi/routing.py Outdated
Comment thread fastapi/routing.py Outdated
@cikay
Copy link
Copy Markdown
Contributor

cikay commented Feb 26, 2022

I suggest squashing the commits. Redundant changes should not be present in repo history as long as it is possible.

@pedroven
Copy link
Copy Markdown
Author

I suggest squashing the commits. Redundant changes should not be present in repo history as long as it is possible.

Done now

@pedroven pedroven changed the title Add exception handler to Unsupported Media Type case Add exception handler to unsupported media type case Feb 26, 2022
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 25, 2022

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (146f57b) 100.00% compared to head (8907282) 99.96%.
Report is 1499 commits behind head on master.

Files Patch % Lines
fastapi/exceptions.py 50.00% 2 Missing ⚠️
fastapi/routing.py 84.61% 2 Missing ⚠️
fastapi/exception_handlers.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##            master    #4615      +/-   ##
===========================================
- Coverage   100.00%   99.96%   -0.04%     
===========================================
  Files          531      531              
  Lines        13627    13646      +19     
===========================================
+ Hits         13627    13641      +14     
- Misses           0        5       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions
Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 18ec2a4 at: https://6266d79fa64e410305a384e9--fastapi.netlify.app

@github-actions
Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 8907282 at: https://626c5d4e8c028500941729e8--fastapi.netlify.app

@gcq
Copy link
Copy Markdown

gcq commented Mar 8, 2023

Hey! Any update on the progress of this PR?

@tiangolo tiangolo added feature New feature or request investigate labels Oct 2, 2023
@tiangolo tiangolo added p3 and removed investigate labels Jan 15, 2024
@pedroven pedroven closed this Mar 28, 2024
@bnawras
Copy link
Copy Markdown

bnawras commented Feb 28, 2025

Hi @tiangolo @cikay,
Could you please share the reasoning behind the decision to ignore this issue?
I’m curious if this is related to current project priorities or if there’s another factor at play.

Thanks!

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

Labels

feature New feature or request p3

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Request with a wrong content-type returns obscure error instead of 415 Unsupported Media Type

7 participants