apig-wsgi
apig-wsgi copied to clipboard
Treat "application/problem+json" as non binary by default
Tests are green locally and seems like this area has test coverage, but please let me know if you'd like me to change parameters/add more test cases
Where does this come from? I need an issue before a random patch.
- Thanks for the quick response!
- I'll open up an issue and connect it to the PR if you want me to (I thought this would improve the code base, there wasn't a bug in the code)
- This hasn't caused an issue on our end, but we thought other users of the repository would benefit from it, as the
mime-typeused in connexion isapplication/problem+json, which is often used to convey errors in a HTTP response
If it hasn't caused any issues, I'd rather do nothing. Especially since it sounds like you haven't tested the change in production..?
We effectively tested the change in production as we're propagating a list of all the defaults and application/problem+json to non_binary_content_type_prefixes when creating our lambda handler using make_lambda_handler