You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The execute() method of MandrillRequestDispatcher has a bug: If the request to Mandrill fails for some reason, the response object gets null. This in turn leads to a NPE in the finally block. When the method is finally left with the NPE, the original exception cause is gone - making debugging very hard.
This codesmell is also detected by SonarLint, by the way.
The
execute()
method ofMandrillRequestDispatcher
has a bug: If the request to Mandrill fails for some reason, theresponse
object gets null. This in turn leads to a NPE in the finally block. When the method is finally left with the NPE, the original exception cause is gone - making debugging very hard.This codesmell is also detected by SonarLint, by the way.
PR is incoming.
See also: https://next.sonarqube.com/sonarqube/coding_rules#rule_key=csharpsquid%3AS1163
The text was updated successfully, but these errors were encountered: