-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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.
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
#749 Bad error handling for IOException
while reading incoming request body
#1769
#749 Bad error handling for IOException
while reading incoming request body
#1769
Conversation
@raman-m I just added a skippable fact, since httpsys is compatible with Windows platform only. |
Please, resolve conflicts! |
78a5c54
to
d65d9ac
Compare
@raman-m you can review it, code is not great, but it's working. I propose reviewing the exception handling later... |
@ggnaegi Please resolve conflicts and rebase onto develop! |
@raman-m Yes, it can be part of Jan 24 Release |
@ggnaegi Please resolve conflicts! |
…two acceptance tests for kestrel and http sys.
…of the exception handling is needed.
629ccb2
to
21113db
Compare
IOException
while reading incoming request body
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ready for delivery! ✅
- Code review ✔️
- Team approvals ✔️
- Unit testing ✔️
- Acceptance tests ✔️✔️
Fixes #749
The error handling in ocelot is hiding the 413 payload too large exception. As @raman-m pointed out, a deeper error handling review is needed. Here, I have implemented a quick fix to this issue and I would recommend opening a new issue for the error handling review.
Proposed Changes
BadHttpRequestException
and if status code == 413, then returning aPayloadTooLargeError