Skip to content

Improve Duo Code Review error messages

Wanderson Policarporequested to merge
580669-improve-duo-code-review-error-messages into master

What does this MR do and why?

Replace the Duo Code Review generic error message where possible with a more detailed message for each case to allow us to track down the root cause for the issue described in Duo Code Review failure after requesting a revi... (#580669).

The generic error message is too broad and is used in several situations outside an exception handling context, which can mask an underlying issue making it silently fail.

References

Related to #580669

How to set up and validate locally

The tests cover pretty much all scenarios, but if you want to validate them locally, the best thing to do is to manipulate the code to force a specific error message.

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Wanderson Policarpo

Merge request reports