Don't disguise important errors with HTTPExceptionDisguise
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
OpenStack Heat |
Triaged
|
Low
|
Kent Wang |
Bug Description
Hi,
The https:/
...
except webob.exc.
if isinstance(err, (webob.exc.HTTPOk, webob.exc.
# Some HTTPException are actually not errors, they are
# responses ready to be sent back to the users, so we don't
# error log, disguise or translate those
raise exception.
My suggestion is to use return with exc. HTTPBadRequest instead of raise in https:/
If this bug is confirmed i will be glad to send you the patch that i have created to fix this.
Thanks for your time.
Changed in heat: | |
assignee: | nobody → Kent Wang (k.wang) |
Changed in heat: | |
milestone: | none → no-priority-tag-bugs |
If you have a patch, please either attach it or propose it via gerrit, it's pointless to say you have a patch and not provide us with visibility of it IMHO, thanks!