Do some cleanup on exception handling.
Review Request #12083 — Created Feb. 24, 2022 and submitted — Latest diff uploaded
Information | |
---|---|
david | |
Review Board | |
release-5.0.x | |
Reviewers | |
reviewboard | |
This change fixes some mechanical issues with exception handling:
- We had a few places that had bare
except
syntax. These have been
updated to explicitly list exception types. - A number of places were assigning the exception to
e
but not using
it. - In one place, we had two handlers for
HostingServiceException
, which
meant that the second one was dead code.
Ran unit tests.