Merge lp:~ack/txaws/catch-all-server-error into lp:txaws
Proposed by
Alberto Donato
Status: | Merged |
---|---|
Merged at revision: | 157 |
Proposed branch: | lp:~ack/txaws/catch-all-server-error |
Merge into: | lp:txaws |
Prerequisite: | lp:~ack/txaws/xss-hardening |
Diff against target: |
45 lines (+9/-6) 2 files modified
txaws/server/resource.py (+5/-1) txaws/server/tests/test_resource.py (+4/-5) |
To merge this branch: | bzr merge lp:~ack/txaws/catch-all-server-error |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Fernando Correa Neto (community) | Approve | ||
txAWS Committers | Pending | ||
Review via email: mp+181563@code.launchpad.net |
Description of the change
This returns a generic "Server error" message for exceptions that are not APIError (the code is already 500 for those responses).
This avoids potential disclosure of server-side information (such as from tracebacks).
To post a comment you must log in.
+1