Skip to content

fix: 전체 로그 포맷 event=* key=value 형식으로 통일#25

Merged
ThinkKat merged 2 commits intomainfrom
dev
Mar 1, 2026
Merged

fix: 전체 로그 포맷 event=* key=value 형식으로 통일#25
ThinkKat merged 2 commits intomainfrom
dev

Conversation

@ThinkKat
Copy link
Copy Markdown
Owner

@ThinkKat ThinkKat commented Mar 1, 2026

Summary

  • PresignedUrlService: 불필요한 INFO 로그 삭제 및 error 로그 event= 포맷 적용
  • GlobalExceptionHandler: 자유 형식 로그 → event= 포맷 통일, PresignedUrlException 중복 로그 제거
  • UserCleanupScheduler: [UserCleanup] 접두사 → event= 포맷 통일

변경 상세

PresignedUrlService

  • generateUploadUrl / generateDownloadUrl INFO 로그 삭제 (상위 서비스 이벤트에서 커버)
  • event=presigned_upload_url_fail, event=presigned_download_url_fail, event=s3_check_fail 적용
  • checkFileExists debug 로그 유지

GlobalExceptionHandler

  • PresignedUrlException 핸들러 로그 삭제 (PresignedUrlService에서 중복 로깅)
  • event=resource_not_found, event=validation_error, event=token_expired, event=unauthorized_access, event=invalid_input, event=unexpected_error 적용

UserCleanupScheduler

  • event=scheduler_user_deleted, event=scheduler_user_cleanup_done, event=scheduler_token_cleanup_done 적용
  • 대상 없을 때 불필요한 INFO 로그 제거

Test plan

  • 업로드 Presigned URL 생성 실패 시 event=presigned_upload_url_fail 로그 확인
  • 존재하지 않는 리소스 접근 시 event=resource_not_found 로그 확인
  • 예상치 못한 예외 발생 시 event=unexpected_error + 스택트레이스 확인

ThinkKat added 2 commits March 1, 2026 21:25
- PresignedUrlService: 불필요한 INFO 로그 삭제, error → event= 포맷
  (generateUploadUrl/downloadUrl INFO 제거, presigned_upload_url_fail,
  presigned_download_url_fail, s3_check_fail 이벤트 코드 적용)
- GlobalExceptionHandler: 자유 형식 → event= 포맷 통일
  (PresignedUrlException 중복 로그 제거, resource_not_found,
  validation_error, token_expired, unauthorized_access,
  invalid_input, unexpected_error 이벤트 코드 적용)
- UserCleanupScheduler: [UserCleanup] 접두사 → event= 포맷 통일
  (scheduler_user_deleted, scheduler_user_cleanup_done,
  scheduler_token_cleanup_done, 대상 없을 때 불필요 INFO 제거)
@ThinkKat ThinkKat merged commit 718851c into main Mar 1, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant