Skip to content

Rescue OpenSSL::SSL::SSLError in JiraService

username-removed-444214 requested to merge mrchrisw-catch-openssl into master

What does this MR do?

Catches OpenSSL::SSL::SSLError in JiraService.

Are there points in the code the reviewer needs to double check?

So... we probably need specs here, but as we're not doing anything with these exceptions (except logging), there's really no specs to add. This is technically poor error handling.

Why was this MR needed?

We blow up if we run into a OpenSSL::SSL::SSLError exception when testing the JiraService.

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Edited by username-removed-444214

Merge request reports

Loading