Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix empty message about raised exception#1815

Merged
tiangolo merged 1 commit intofastapi:masterfrom
manlix:handling_errors
Aug 9, 2020
Merged

Fix empty message about raised exception#1815
tiangolo merged 1 commit intofastapi:masterfrom
manlix:handling_errors

Conversation

@manlix
Copy link
Contributor

Fix for issue in documentation example code#1814

@codecov
Copy link

codecovbot commentedJul 30, 2020
edited
Loading

Codecov Report

Merging#1815 intomaster willnot change coverage.
The diff coverage isn/a.

Impacted file tree graph

@@            Coverage Diff             @@##            master     #1815    +/-   ##==========================================  Coverage   100.00%   100.00%            ==========================================  Files          235       235              Lines         6989      6833   -156     ==========================================- Hits          6989      6833   -156
Impacted FilesCoverage Δ
tests/test_path.py100.00% <0.00%> (ø)
tests/test_query.py100.00% <0.00%> (ø)
tests/test_fakeasync.py100.00% <0.00%> (ø)
tests/test_application.py100.00% <0.00%> (ø)
tests/test_params_repr.py100.00% <0.00%> (ø)
tests/test_empty_router.py100.00% <0.00%> (ø)
tests/test_router_events.py100.00% <0.00%> (ø)
tests/test_openapi_servers.py100.00% <0.00%> (ø)
tests/test_security_oauth2.py100.00% <0.00%> (ø)
tests/test_dependency_class.py100.00% <0.00%> (ø)
... and56 more

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last updatea63b1ef...df17f57. Read thecomment docs.

@github-actions
Copy link
Contributor

📝 Docs preview for commitdf17f57 at:https://5f22e34b127c7c469e7479bd--fastapi.netlify.app

Copy link
Member

@KludexKludex left a comment
edited
Loading

Choose a reason for hiding this comment

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

Do we have the same problem on next exception handler (RequestValidationError)?

But nice catch! 🎉

@manlix
Copy link
ContributorAuthor

Do we have the same problem on next exception handler (RequestValidationError)?

But nice catch!

@Kludex No, we don't. Next exception handler is correct:

manlix@lab:~$ curl http://localhost:8000/items/incorrect{"detail":[{"loc":["path","item_id"],"msg":"value is not a valid integer","type":"type_error.integer"}]}

Output in logs has expected details in plain text:

OMG! The client sent invalid data!: 1 validation errorfor Requestpath -> item_id  value is not a valid integer (type=type_error.integer)INFO:     127.0.0.1:50902 -"GET /items/incorrect HTTP/1.1" 422 Unprocessable Entity
Kludex reacted with rocket emoji

@tiangolotiangolo merged commit0752c72 intofastapi:masterAug 9, 2020
@tiangolo
Copy link
Member

Awesome, good catch@manlix ! 🚀 🍰

Thanks for the help@Kludex ! ☕

Kludex reacted with hooray emoji

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

Reviewers

@KludexKludexKludex approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@manlix@tiangolo@Kludex

Comments


[8]ページ先頭

©2009-2026 Movatter.jp