adding log message in app service and controller class

ministryofjustice · Mar 5, 2025 · 2479200 · 2479200
1 parent 1b1387f
commit 2479200
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 0 deletions.
diff --git a/...ain/kotlin/uk/gov/justice/digital/hmpps/managingprisonerappsapi/resource/AppController.kt b/...ain/kotlin/uk/gov/justice/digital/hmpps/managingprisonerappsapi/resource/AppController.kt
@@ -1,5 +1,6 @@
 package uk.gov.justice.digital.hmpps.managingprisonerappsapi.resource
 
+import org.slf4j.LoggerFactory
 import org.springframework.http.HttpStatus
 import org.springframework.http.MediaType
 import org.springframework.http.ResponseEntity
@@ -22,6 +23,10 @@ import java.util.*
 @RequestMapping("v1")
 class AppController(var appService: AppService) {
 
+  companion object {
+    private val logger = LoggerFactory.getLogger(AppController::class.java)
+  }
+
   @PostMapping(
     "prisoners/{prisoner-id}/apps",
     produces = [MediaType.APPLICATION_JSON_VALUE],
@@ -34,6 +39,7 @@ class AppController(var appService: AppService) {
     authentication: Authentication,
   ): ResponseEntity<AppResponseDto> {
     authentication as AuthAwareAuthenticationToken
+    logger.info("Request received for submitting app for $prisonerId by ${authentication.principal}")
     val appResponseDto = appService.submitApp(prisonerId, authentication.principal, appRequestDto)
     return ResponseEntity.status(HttpStatus.CREATED).body(appResponseDto)
   }
@@ -49,6 +55,8 @@ class AppController(var appService: AppService) {
     @RequestParam(required = false) assignedGroup: Boolean,
     authentication: Authentication,
   ): ResponseEntity<AppResponseDto> {
+    authentication as AuthAwareAuthenticationToken
+    logger.info("Request received for get app for $prisonerId by ${authentication.principal}")
     val appResponseDto = appService.getAppsById(prisonerId, id, requestedBy, assignedGroup)
     return ResponseEntity.status(HttpStatus.OK).body(appResponseDto)
   }

diff --git a/...ain/kotlin/uk/gov/justice/digital/hmpps/managingprisonerappsapi/service/AppServiceImpl.kt b/...ain/kotlin/uk/gov/justice/digital/hmpps/managingprisonerappsapi/service/AppServiceImpl.kt
@@ -1,5 +1,6 @@
 package uk.gov.justice.digital.hmpps.managingprisonerappsapi.service
 
+import org.slf4j.LoggerFactory
 import org.springframework.http.HttpStatus
 import org.springframework.stereotype.Service
 import uk.gov.justice.digital.hmpps.managingprisonerappsapi.dto.AppRequestDto
@@ -12,6 +13,7 @@ import uk.gov.justice.digital.hmpps.managingprisonerappsapi.model.AppType
 import uk.gov.justice.digital.hmpps.managingprisonerappsapi.model.Prisoner
 import uk.gov.justice.digital.hmpps.managingprisonerappsapi.model.Staff
 import uk.gov.justice.digital.hmpps.managingprisonerappsapi.repository.AppRepository
+import uk.gov.justice.digital.hmpps.managingprisonerappsapi.resource.AppController
 import java.time.LocalDateTime
 import java.util.*
 
@@ -22,6 +24,10 @@ class AppServiceImpl(
   var staffService: StaffService,
 ) : AppService {
 
+  companion object {
+    private val logger = LoggerFactory.getLogger(AppController::class.java)
+  }
+
   fun saveApp(app: App): App = appRepository.save(app)
 
   fun updateApp(app: App): App = appRepository.save(app)
@@ -57,6 +63,7 @@ class AppServiceImpl(
       null,
       null,
     )
+    logger.info("App created for $prisonerId for app type ${app.appType}")
     return convertAppToAppResponseDto(prisonerId, app, prisonerId, assignedGroup)
   }