Improve error information when the GitLab API version can't be found.

Review Request #10336 — Created Nov. 27, 2018 and submitted — Latest diff uploaded

Information

Review Board
release-3.0.x
4eb5bed...

Reviewers

Users working with self-hosted GitLab installs sometimes hit errors
saying the GitLab API version could not be found. While we attempt to
store the reasons this may occur, we never actually output them
anywhere, and when authenticating against the repository we prefix with
a redundant "Unknown error" and stack trace, which doesn't help matters
too much.

This redoes this error reporting to provide a more helpful, informative
error message that lists the causes we stored (hopefully providing some
context to this error), along with some helpful hints on what may have
gone wrong (URL resolution issues or SSL certificate trust issues).
They're no longer shown as unknown errors, and the logs aren't full of
scary stack traces.

The generic error handler for the repository form has also received some
very minor tweaks, saying "Unexpected error" instead of "Uknown error"
and saying that the log file "may" have additional details. It also
removes a logging stack trace, which became redundant with the addition
of a similar logging statement in HostingServiceAuthForm.authorize
when generic SSL cert verification was introduced.

Added new unit tests, which pass.

Manually tested using some bad URLs and saw the more reasonable errors
without all the redundancy.

    Loading...