ref #78

Author: arthurgregorio

src/main/kotlin/br/com/webbudget/application/controllers/advice/ExceptionHandlerAdvice.kt

@@ -1,6 +1,7 @@
 package br.com.webbudget.application.controllers.advice
 
 import br.com.webbudget.domain.exceptions.BusinessException
+import io.github.oshai.kotlinlogging.KotlinLogging
 import jakarta.servlet.http.HttpServletResponse
 import org.springframework.dao.NonTransientDataAccessException
 import org.springframework.http.HttpStatus
@@ -10,6 +11,8 @@ import org.springframework.security.authentication.BadCredentialsException
 import org.springframework.web.bind.annotation.ExceptionHandler
 import org.springframework.web.bind.annotation.RestControllerAdvice
 
+private val logger = KotlinLogging.logger {}
+
 @RestControllerAdvice
 class ExceptionHandlerAdvice {
 
@@ -19,16 +22,22 @@ class ExceptionHandlerAdvice {
     }
 
     @ExceptionHandler(IllegalArgumentException::class)
-    fun handle(ex: IllegalArgumentException): ProblemDetail =
-        ProblemDetail.forStatusAndDetail(BAD_REQUEST, ex.message ?: NO_DETAIL_PROVIDED)
+    fun handle(ex: IllegalArgumentException): ProblemDetail {
+        logger.error(ex) { "Illegal argument exception" }
+        return ProblemDetail.forStatusAndDetail(BAD_REQUEST, ex.message ?: NO_DETAIL_PROVIDED)
+    }
 
     @ExceptionHandler(NonTransientDataAccessException::class)
-    fun handle(ex: NonTransientDataAccessException): ProblemDetail =
-        ProblemDetail.forStatusAndDetail(BAD_REQUEST, ex.message ?: NO_DETAIL_PROVIDED)
+    fun handle(ex: NonTransientDataAccessException): ProblemDetail {
+        logger.error(ex) { "Non transient data access exception" }
+        return ProblemDetail.forStatusAndDetail(BAD_REQUEST, ex.message ?: NO_DETAIL_PROVIDED)
+    }
 
     @ExceptionHandler(BusinessException::class)
-    fun handle(ex: BusinessException): ProblemDetail =
-        ProblemDetail.forStatusAndDetail(ex.httpStatus, ex.message ?: NO_DETAIL_PROVIDED)
+    fun handle(ex: BusinessException): ProblemDetail {
+        logger.error(ex) { "Business exception" }
+        return ProblemDetail.forStatusAndDetail(ex.httpStatus, ex.message ?: NO_DETAIL_PROVIDED)
+    }
 
     companion object {
         private const val NO_DETAIL_PROVIDED = "No detail message provided"