-
Notifications
You must be signed in to change notification settings - Fork 421
Feature request: Additional HTTP Error Code Exceptions #6379
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
Comments
Thanks for opening your first issue here! We'll come back to you as soon as we can. |
Hello @kazu728 ! Would you like to submit a PR with these changes? |
Thank you for deaIing with this issue. I do not suggest any other changes.
Yes, I will submit it. |
Hi @kazu728! Please let me know if you have time to submit a PR in the next 2 weeks. I'm not pushing, but since we are planing the release, we'd like to add this. I totally understand if you can't, we're all busy. |
Sorry, I will deal with this issue by the weekend. |
Hey @kazu728! Thanks a lot. |
|
Use case
ref: https://github.com/aws-powertools/powertools-lambda-python/blob/develop/aws_lambda_powertools/event_handler/exceptions.py
I need to handle additional HTTP error codes, specifically 413 (Payload Too Large). I notice that the current exceptions.py only implements a subset of HTTP error codes (400, 401, 404, 500).
While I understand I can extend ServiceError to create my own custom exceptions, this creates inconsistency in my codebase where some errors come from PowerTools and others from my custom definitions.
Are there plans to expand the list of predefined exceptions to cover more HTTP status codes?
What is the rationale behind which HTTP errors are currently implemented vs. those that aren't?
Solution/User Experience
Add status code
Alternative solutions
Acknowledgment
The text was updated successfully, but these errors were encountered: