generated from pricelees/issue-pr-template
refactor: MemberId MDC 키 상수화
This commit is contained in:
parent
ef58752cec
commit
9f1b13a1ea
@ -16,6 +16,8 @@ import roomescape.member.infrastructure.persistence.MemberEntity
|
||||
|
||||
private val log: KLogger = KotlinLogging.logger {}
|
||||
|
||||
const val MDC_MEMBER_ID_KEY: String = "member_id"
|
||||
|
||||
@Component
|
||||
class AuthInterceptor(
|
||||
private val memberFinder: MemberFinder,
|
||||
@ -42,7 +44,7 @@ class AuthInterceptor(
|
||||
throw AuthException(AuthErrorCode.ACCESS_DENIED)
|
||||
}
|
||||
|
||||
MDC.put("member_id", "${member.id}")
|
||||
MDC.put(MDC_MEMBER_ID_KEY, "${member.id}")
|
||||
log.info { "[AuthInterceptor] 인증 완료. memberId=${member.id}, role=${member.role}" }
|
||||
return true
|
||||
}
|
||||
@ -51,7 +53,7 @@ class AuthInterceptor(
|
||||
try {
|
||||
val memberId = jwtHandler.getMemberIdFromToken(accessToken)
|
||||
return memberFinder.findById(memberId)
|
||||
.also { MDC.put("member_id", "$memberId") }
|
||||
.also { MDC.put(MDC_MEMBER_ID_KEY, "$memberId") }
|
||||
} catch (e: Exception) {
|
||||
log.info { "[AuthInterceptor] 회원 조회 실패. accessToken = $accessToken" }
|
||||
val errorCode = AuthErrorCode.MEMBER_NOT_FOUND
|
||||
|
||||
@ -3,6 +3,7 @@ package roomescape.common.log
|
||||
import com.fasterxml.jackson.databind.ObjectMapper
|
||||
import jakarta.servlet.http.HttpServletRequest
|
||||
import org.slf4j.MDC
|
||||
import roomescape.auth.web.support.MDC_MEMBER_ID_KEY
|
||||
|
||||
enum class LogType {
|
||||
INCOMING_HTTP_REQUEST,
|
||||
@ -33,7 +34,7 @@ class ApiLogMessageConverter(
|
||||
controllerPayload: Map<String, Any>,
|
||||
): String {
|
||||
val payload: MutableMap<String, Any> = commonRequestPayload(LogType.CONTROLLER_INVOKED, request)
|
||||
val memberId: Long? = MDC.get("member_id")?.toLong()
|
||||
val memberId: Long? = MDC.get(MDC_MEMBER_ID_KEY)?.toLong()
|
||||
if (memberId != null) payload["member_id"] = memberId else payload["member_id"] = "NONE"
|
||||
|
||||
payload.putAll(controllerPayload)
|
||||
@ -46,7 +47,7 @@ class ApiLogMessageConverter(
|
||||
payload["type"] = request.type
|
||||
payload["status_code"] = request.httpStatus
|
||||
|
||||
MDC.get("member_id")?.toLongOrNull()
|
||||
MDC.get(MDC_MEMBER_ID_KEY)?.toLongOrNull()
|
||||
?.let { payload["member_id"] = it }
|
||||
?: run { payload["member_id"] = "NONE" }
|
||||
|
||||
|
||||
Loading…
x
Reference in New Issue
Block a user