CODEWITHSUNDEEP
spring-bootmavensonarqube
Krish
Krish

Reputation: 4232

Sonar complaining logging or rethrowing the exception

I have the following below piece of code when I am running SonarQube for code quality check on it after integrating it with Maven.

However, Sonar is complaining that I should Either log or rethrow this exception.

What am I missing here? Can some one help me please.

Code

public ShippingResponse processShipping(ShippingRequest request) {
        log.debug("Processing Reservation Request ....");
        try{
            return helper.processShippingMethod(request);
        } catch (ServiceException serviceException) {
            log.error(RESERVATION_EXCE, ExceptionUtils.getStackTrace(serviceException));
            throw serviceException;
        } catch (Exception e) {
            throw new ServiceException(ErrorMessages.EPO_SM_ERR_03, e.getMessage());
        }
    }

enter image description here

Upvotes: 0

Views: 854

Answers (1)

Nico Van Belle
Nico Van Belle

Reputation: 5146

The point that Sonar is trying to make is that you ideally print or keep the root cause of your exception, so basically the stack. You keep it by passing the exception object because if you only keep the message you lose all that information. To make sonar happy you either print the stack trace (log.error(ErrorMessages.EPO_SM_ERR_03, e)), or re-throw a new exception passing the Throwable object to the constructor.

So the ideal solution would be to use the ServiceException like this;

public class ServiceException extends Exception {
    public ServiceException(String message, Throwable cause) {
        super(message, cause);
    }
}

throw new ServiceException(ErrorMessages.EPO_SM_ERR_03, e);

Upvotes: 1

Related Questions