feat: file logger

src/main/kotlin/com/tistory/shanepark/dutypark/common/advice/ViewExceptionControllerAdvice.kt

@@ -2,6 +2,7 @@ package com.tistory.shanepark.dutypark.common.advice
 
 import com.tistory.shanepark.dutypark.common.exceptions.DutyparkAuthException
 import jakarta.servlet.http.HttpServletRequest
+import org.slf4j.LoggerFactory
 import org.springframework.core.Ordered
 import org.springframework.core.annotation.Order
 import org.springframework.stereotype.Controller
@@ -13,7 +14,7 @@ import java.net.URLEncoder
 @ControllerAdvice(annotations = [Controller::class])
 @Order(Ordered.HIGHEST_PRECEDENCE)
 class ViewExceptionControllerAdvice {
-    val log: org.slf4j.Logger = org.slf4j.LoggerFactory.getLogger(this.javaClass)
+    val log: org.slf4j.Logger = LoggerFactory.getLogger(this.javaClass)
 
     @ExceptionHandler
     fun notAuthorizedHandler(e: DutyparkAuthException, request: HttpServletRequest): ModelAndView {

src/main/kotlin/com/tistory/shanepark/dutypark/common/config/LogbackConfig.kt

@@ -0,0 +1,57 @@
+package com.tistory.shanepark.dutypark.common.config
+
+import ch.qos.logback.classic.Level
+import ch.qos.logback.classic.LoggerContext
+import ch.qos.logback.classic.encoder.PatternLayoutEncoder
+import ch.qos.logback.classic.spi.ILoggingEvent
+import ch.qos.logback.core.rolling.RollingFileAppender
+import ch.qos.logback.core.rolling.TimeBasedRollingPolicy
+import jakarta.annotation.PostConstruct
+import org.slf4j.Logger
+import org.slf4j.LoggerFactory
+import org.springframework.beans.factory.annotation.Value
+import org.springframework.context.annotation.Configuration
+
+@Configuration
+class LogbackConfig {
+
+    @Value("\${dutypark.log.path}")
+    private lateinit var logPath: String
+
+    val log: Logger = LoggerFactory.getLogger(LogbackConfig::class.java)
+
+    @PostConstruct
+    fun configureLogging() {
+        val loggerContext = LoggerFactory.getILoggerFactory() as LoggerContext
+
+        val encoder = PatternLayoutEncoder().apply {
+            context = loggerContext
+            pattern = "%d{yyyy-MM-dd HH:mm:ss} [%thread] %-5level %logger{36} - %msg%n"
+            start()
+        }
+
+        val fileAppender = RollingFileAppender<ILoggingEvent>().apply {
+            context = loggerContext
+            name = "FILE"
+            this.encoder = encoder
+            isAppend = true
+        }
+
+        val rollingPolicy = TimeBasedRollingPolicy<ILoggingEvent>().apply {
+            context = loggerContext
+            fileNamePattern = "${logPath}/dutypark-%d{yyyy-MM-dd}.log"
+            maxHistory = 365
+            setParent(fileAppender)
+            start()
+        }
+
+        fileAppender.rollingPolicy = rollingPolicy
+        fileAppender.start()
+
+        val rootLogger = loggerContext.getLogger("ROOT")
+        rootLogger.level = Level.INFO
+        rootLogger.addAppender(fileAppender)
+
+        log.info("log file path: $logPath")
+    }
+}

src/main/kotlin/com/tistory/shanepark/dutypark/common/slack/SlackApiConfiguration.kt

@@ -6,6 +6,7 @@ import com.tistory.shanepark.dutypark.common.slack.notifier.SlackNotifierSender
 import net.gpedro.integrations.slack.SlackApi
 import net.gpedro.integrations.slack.SlackMessage
 import org.slf4j.Logger
+import org.slf4j.LoggerFactory
 import org.springframework.beans.factory.annotation.Value
 import org.springframework.context.annotation.Bean
 import org.springframework.context.annotation.Configuration
@@ -19,7 +20,7 @@ class SlackApiConfiguration(
     val slackToken: String
 ) {
 
-    private val log: Logger = org.slf4j.LoggerFactory.getLogger(this.javaClass)
+    private val log: Logger = LoggerFactory.getLogger(this.javaClass)
 
     @Bean("slackTaskExecutor")
     fun threadPoolTaskExecutor(): TaskExecutor {

src/main/kotlin/com/tistory/shanepark/dutypark/common/slack/advice/ErrorDetectAdvisor.kt

@@ -8,6 +8,7 @@ import net.gpedro.integrations.slack.SlackMessage
 import org.apache.catalina.connector.ClientAbortException
 import org.apache.coyote.CloseNowException
 import org.slf4j.Logger
+import org.slf4j.LoggerFactory
 import org.springframework.http.ResponseEntity
 import org.springframework.web.HttpRequestMethodNotSupportedException
 import org.springframework.web.bind.annotation.ControllerAdvice
@@ -21,7 +22,7 @@ import java.util.*
 class ErrorDetectAdvisor(
     private val slackNotifier: SlackNotifier,
 ) {
-    val log: Logger = org.slf4j.LoggerFactory.getLogger(this.javaClass)
+    val log: Logger = LoggerFactory.getLogger(this.javaClass)
 
     @ExceptionHandler(HttpRequestMethodNotSupportedException::class)
     @ResponseBody

src/main/kotlin/com/tistory/shanepark/dutypark/common/slack/notifier/SlackNotifierLogger.kt

@@ -1,10 +1,11 @@
 package com.tistory.shanepark.dutypark.common.slack.notifier
 
 import net.gpedro.integrations.slack.SlackMessage
+import org.slf4j.LoggerFactory
 
 class SlackNotifierLogger : SlackNotifier {
 
-    private val log: org.slf4j.Logger = org.slf4j.LoggerFactory.getLogger(this.javaClass)
+    private val log: org.slf4j.Logger = LoggerFactory.getLogger(this.javaClass)
 
     override fun call(slackMessage: SlackMessage) {
         log.info("SlackNotifierLogger: $slackMessage")

src/main/kotlin/com/tistory/shanepark/dutypark/department/controller/DepartmentAdminController.kt

@@ -1,6 +1,7 @@
 package com.tistory.shanepark.dutypark.department.controller
 
 import com.tistory.shanepark.dutypark.common.domain.dto.PageResponse
+import com.tistory.shanepark.dutypark.common.exceptions.DutyparkAuthException
 import com.tistory.shanepark.dutypark.dashboard.domain.DashboardDepartment
 import com.tistory.shanepark.dutypark.department.domain.dto.DepartmentCreateDto
 import com.tistory.shanepark.dutypark.department.domain.dto.DepartmentDto
@@ -13,8 +14,12 @@ import com.tistory.shanepark.dutypark.duty.batch.domain.DutyBatchTeamResult
 import com.tistory.shanepark.dutypark.duty.batch.domain.DutyBatchTemplate
 import com.tistory.shanepark.dutypark.duty.batch.exceptions.DutyBatchException
 import com.tistory.shanepark.dutypark.duty.batch.service.DutyBatchService
+import com.tistory.shanepark.dutypark.member.domain.annotation.Login
 import com.tistory.shanepark.dutypark.member.repository.MemberRepository
+import com.tistory.shanepark.dutypark.security.domain.dto.LoginMember
 import jakarta.validation.Valid
+import org.slf4j.Logger
+import org.slf4j.LoggerFactory
 import org.springframework.context.ApplicationContext
 import org.springframework.data.domain.Pageable
 import org.springframework.data.web.PageableDefault
@@ -32,6 +37,8 @@ class DepartmentAdminController(
     private val applicationContext: ApplicationContext,
 ) {
 
+    val log: Logger = LoggerFactory.getLogger(DepartmentAdminController::class.java)
+
     @GetMapping
     fun findAll(@PageableDefault(page = 0, size = 10) page: Pageable): PageResponse<SimpleDepartmentDto> {
         val result = departmentService.findAllWithMemberCount(page)
@@ -90,22 +97,27 @@ class DepartmentAdminController(
 
     @PostMapping("/{id}/duty")
     fun uploadBatchTemplate(
+        @Login loginMember: LoginMember,
         @PathVariable id: Long,
         @RequestParam(name = "file") file: MultipartFile,
         @RequestParam(name = "year") year: Int,
         @RequestParam(name = "month") month: Int
     ): DutyBatchTeamResult {
+        if (!loginMember.isAdmin) {
+            throw DutyparkAuthException("$loginMember is not admin")
+        }
         val department = departmentRepository.findById(id).orElseThrow()
         val batchTemplate = department.dutyBatchTemplate ?: throw IllegalArgumentException("templateName is required")
         val dutyBatchService = applicationContext.getBean(batchTemplate.batchServiceClass) as DutyBatchService
-        try {
-            return dutyBatchService.batchUploadDepartment(
+        return try {
+            log.info("batch duty upload by $loginMember for department ${department.name}(${department.id}) year=$year, month=$month")
+            dutyBatchService.batchUploadDepartment(
                 departmentId = id,
                 file = file,
                 yearMonth = YearMonth.of(year, month)
             )
         } catch (e: DutyBatchException) {
-            return DutyBatchTeamResult.fail(e.message ?: "알 수 없는 원인으로 시간표 업로드 실패.")
+            DutyBatchTeamResult.fail(e.message ?: "알 수 없는 원인으로 시간표 업로드 실패.")
         }
     }
 

src/main/kotlin/com/tistory/shanepark/dutypark/duty/controller/DutyBatchController.kt

@@ -11,6 +11,7 @@ import com.tistory.shanepark.dutypark.member.domain.annotation.Login
 import com.tistory.shanepark.dutypark.member.service.MemberService
 import com.tistory.shanepark.dutypark.security.domain.dto.LoginMember
 import org.slf4j.Logger
+import org.slf4j.LoggerFactory
 import org.springframework.context.ApplicationContext
 import org.springframework.web.bind.annotation.*
 import org.springframework.web.multipart.MultipartFile
@@ -23,7 +24,7 @@ class DutyBatchController(
     private val memberService: MemberService,
     private val applicationContext: ApplicationContext,
 ) {
-    val log: Logger = org.slf4j.LoggerFactory.getLogger(DutyBatchController::class.java)
+    val log: Logger = LoggerFactory.getLogger(DutyBatchController::class.java)
 
     @GetMapping("/templates")
     fun getTemplates(
@@ -48,6 +49,7 @@ class DutyBatchController(
 
         val dutyBatchService = applicationContext.getBean(dutyBatchTemplate.batchServiceClass) as DutyBatchService
         return try {
+            log.info("batch duty upload by $loginMember for member $memberId. year=$year, month=$month")
             dutyBatchService.batchUploadMember(memberId = memberId, file = file, yearMonth = YearMonth.of(year, month))
         } catch (e: DutyBatchException) {
             DutyBatchResult.fail(e.batchErrorMessage)

src/main/kotlin/com/tistory/shanepark/dutypark/duty/controller/DutyController.kt

@@ -8,6 +8,7 @@ import com.tistory.shanepark.dutypark.duty.service.DutyService
 import com.tistory.shanepark.dutypark.member.domain.annotation.Login
 import com.tistory.shanepark.dutypark.security.domain.dto.LoginMember
 import org.slf4j.Logger
+import org.slf4j.LoggerFactory
 import org.springframework.http.ResponseEntity
 import org.springframework.web.bind.annotation.*
 import java.time.YearMonth
@@ -17,7 +18,7 @@ import java.time.YearMonth
 class DutyController(
     private val dutyService: DutyService,
 ) {
-    val log: Logger = org.slf4j.LoggerFactory.getLogger(DutyController::class.java)
+    val log: Logger = LoggerFactory.getLogger(DutyController::class.java)
 
     @GetMapping
     fun getDuties(
@@ -40,6 +41,7 @@ class DutyController(
     ): ResponseEntity<Boolean> {
         checkAuthentication(loginMember, dutyUpdateDto.memberId)
         dutyService.update(dutyUpdateDto)
+        log.info("$loginMember update duty: $dutyUpdateDto")
         return ResponseEntity.ok(true)
     }
 
@@ -50,6 +52,7 @@ class DutyController(
     ): ResponseEntity<Boolean> {
         checkAuthentication(loginMember, dutyBatchUpdateDto.memberId)
         dutyService.update(dutyBatchUpdateDto)
+        log.info("$loginMember update duty batch: $dutyBatchUpdateDto")
         return ResponseEntity.ok(true)
     }
 

src/main/kotlin/com/tistory/shanepark/dutypark/duty/controller/DutyViewController.kt

@@ -6,6 +6,7 @@ import com.tistory.shanepark.dutypark.member.service.FriendService
 import com.tistory.shanepark.dutypark.member.service.MemberService
 import com.tistory.shanepark.dutypark.security.domain.dto.LoginMember
 import org.slf4j.Logger
+import org.slf4j.LoggerFactory
 import org.springframework.stereotype.Controller
 import org.springframework.ui.Model
 import org.springframework.web.bind.annotation.GetMapping
@@ -18,7 +19,7 @@ class DutyViewController(
     val memberService: MemberService,
     val friendService: FriendService,
 ) : ViewController() {
-    val log: Logger = org.slf4j.LoggerFactory.getLogger(this.javaClass)
+    val log: Logger = LoggerFactory.getLogger(this.javaClass)
 
     @GetMapping("/duty/{id}")
     fun retrieveMemberDuty(

src/main/kotlin/com/tistory/shanepark/dutypark/member/controller/DDayController.kt

@@ -6,6 +6,7 @@ import com.tistory.shanepark.dutypark.member.domain.dto.DDaySaveDto
 import com.tistory.shanepark.dutypark.member.service.DDayService
 import com.tistory.shanepark.dutypark.security.domain.dto.LoginMember
 import jakarta.validation.Valid
+import org.slf4j.LoggerFactory
 import org.springframework.http.HttpStatus
 import org.springframework.http.ResponseEntity
 import org.springframework.web.bind.annotation.*
@@ -15,7 +16,7 @@ import org.springframework.web.bind.annotation.*
 class DDayController(
     private val dDayService: DDayService
 ) {
-    private val log: org.slf4j.Logger = org.slf4j.LoggerFactory.getLogger(this.javaClass)
+    private val log = LoggerFactory.getLogger(this.javaClass)
 
     @PostMapping
     @SlackNotification

src/main/kotlin/com/tistory/shanepark/dutypark/member/service/DDayService.kt

@@ -7,6 +7,8 @@ import com.tistory.shanepark.dutypark.member.domain.entity.DDayEvent
 import com.tistory.shanepark.dutypark.member.repository.DDayRepository
 import com.tistory.shanepark.dutypark.member.repository.MemberRepository
 import com.tistory.shanepark.dutypark.security.domain.dto.LoginMember
+import org.slf4j.Logger
+import org.slf4j.LoggerFactory
 import org.springframework.stereotype.Service
 import org.springframework.transaction.annotation.Transactional
 import java.time.LocalDate
@@ -18,7 +20,7 @@ class DDayService(
     private val dDayRepository: DDayRepository
 ) {
 
-    val log: org.slf4j.Logger = org.slf4j.LoggerFactory.getLogger(this.javaClass)
+    val log: Logger = LoggerFactory.getLogger(this.javaClass)
 
     fun createDDay(loginMember: LoginMember, dDaySaveDto: DDaySaveDto): DDayEvent {
         val member = memberRepository.findById(loginMember.id).orElseThrow()

src/main/kotlin/com/tistory/shanepark/dutypark/member/service/RefreshTokenService.kt

@@ -9,6 +9,7 @@ import com.tistory.shanepark.dutypark.security.domain.dto.LoginMember
 import com.tistory.shanepark.dutypark.security.domain.dto.RefreshTokenDto
 import com.tistory.shanepark.dutypark.security.domain.entity.RefreshToken
 import org.slf4j.Logger
+import org.slf4j.LoggerFactory
 import org.springframework.scheduling.annotation.Scheduled
 import org.springframework.stereotype.Service
 import java.time.LocalDateTime
@@ -19,7 +20,7 @@ class RefreshTokenService(
     private val refreshTokenRepository: RefreshTokenRepository,
     private val jwtConfig: JwtConfig,
 ) {
-    private val log: Logger = org.slf4j.LoggerFactory.getLogger(this.javaClass)
+    private val log: Logger = LoggerFactory.getLogger(this.javaClass)
 
     @Scheduled(cron = "0 0 0 * * *")
     fun revokeExpiredRefreshTokens() {
@@ -39,7 +40,7 @@ class RefreshTokenService(
     fun deleteRefreshToken(loginMember: LoginMember, id: Long) {
         val refreshToken = refreshTokenRepository.findById(id).orElseThrow()
         if (!loginMember.isAdmin && refreshToken.member.id != loginMember.id) {
-            log.warn("No authority to delete refresh token. loginMemberId:${loginMember.id}, refreshTokenId:$id")
+            log.warn("No authority to delete refresh token. loginMemberId:$loginMember, refreshTokenId:$id")
             throw DutyparkAuthException("No authority to delete refresh token.")
         }
         refreshTokenRepository.delete(refreshToken)

src/main/kotlin/com/tistory/shanepark/dutypark/schedule/service/ScheduleService.kt

@@ -12,6 +12,7 @@ import com.tistory.shanepark.dutypark.schedule.domain.enums.ParsingTimeStatus
 import com.tistory.shanepark.dutypark.schedule.repository.ScheduleRepository
 import com.tistory.shanepark.dutypark.schedule.timeparsing.service.ScheduleTimeParsingQueueManager
 import com.tistory.shanepark.dutypark.security.domain.dto.LoginMember
+import org.slf4j.LoggerFactory
 import org.springframework.stereotype.Service
 import org.springframework.transaction.annotation.Transactional
 import java.time.LocalDateTime
@@ -26,7 +27,7 @@ class ScheduleService(
     private val friendService: FriendService,
     private val scheduleTimeParsingQueueManager: ScheduleTimeParsingQueueManager,
 ) {
-    private val log = org.slf4j.LoggerFactory.getLogger(this.javaClass)
+    private val log = LoggerFactory.getLogger(this.javaClass)
 
     @Transactional(readOnly = true)
     fun findSchedulesByYearAndMonth(

src/main/kotlin/com/tistory/shanepark/dutypark/schedule/timeparsing/service/ScheduleTimeParsingService.kt

@@ -4,6 +4,8 @@ import com.fasterxml.jackson.core.JsonProcessingException
 import com.fasterxml.jackson.databind.ObjectMapper
 import com.tistory.shanepark.dutypark.schedule.timeparsing.domain.ScheduleTimeParsingRequest
 import com.tistory.shanepark.dutypark.schedule.timeparsing.domain.ScheduleTimeParsingResponse
+import org.slf4j.Logger
+import org.slf4j.LoggerFactory
 import org.springframework.ai.chat.client.ChatClient
 import org.springframework.ai.chat.model.ChatModel
 import org.springframework.stereotype.Service
@@ -14,7 +16,7 @@ class ScheduleTimeParsingService(
     private val objectMapper: ObjectMapper,
 ) {
     private val chatClient = ChatClient.builder(chatModel).build()
-    private val log: org.slf4j.Logger = org.slf4j.LoggerFactory.getLogger(ScheduleTimeParsingService::class.java)
+    private val log: Logger = LoggerFactory.getLogger(ScheduleTimeParsingService::class.java)
 
     fun parseScheduleTime(request: ScheduleTimeParsingRequest): ScheduleTimeParsingResponse {
         val prompt = generatePrompt(request)

src/main/kotlin/com/tistory/shanepark/dutypark/security/controller/AuthController.kt

@@ -7,6 +7,7 @@ import com.tistory.shanepark.dutypark.security.domain.dto.LoginMember
 import com.tistory.shanepark.dutypark.security.domain.dto.PasswordChangeDto
 import com.tistory.shanepark.dutypark.security.service.AuthService
 import jakarta.servlet.http.HttpServletRequest
+import org.slf4j.LoggerFactory
 import org.springframework.http.ResponseEntity
 import org.springframework.ui.Model
 import org.springframework.web.bind.annotation.*
@@ -16,7 +17,7 @@ import org.springframework.web.bind.annotation.*
 class AuthController(
     private val authService: AuthService,
 ) {
-    private val log = org.slf4j.LoggerFactory.getLogger(AuthController::class.java)
+    private val log = LoggerFactory.getLogger(AuthController::class.java)
 
     @PostMapping("login")
     fun login(

src/main/kotlin/com/tistory/shanepark/dutypark/security/domain/dto/LoginMember.kt

@@ -12,4 +12,9 @@ data class LoginMember(
         const val ATTR_NAME: String = "loginMember"
     }
 
+    override fun toString(): String {
+        return "${name}(${id})"
+    }
+
+
 }