Skip to content

[BUG FIX/90] S3 로그 업로드 로직 수정 및 병합 로직 제거#138

Merged
persi0815 merged 3 commits intodevelopfrom
bugfix/90
Jan 2, 2025
Merged

[BUG FIX/90] S3 로그 업로드 로직 수정 및 병합 로직 제거#138
persi0815 merged 3 commits intodevelopfrom
bugfix/90

Conversation

@persi0815
Copy link
Contributor

@persi0815 persi0815 commented Jan 2, 2025

PR 타입

  • 버그 수정

구현한 기능

  • S3와 로컬 로그 병합 로직 제거로 간소화.
  • 업로드 완료 후 로컬 로그 파일 자동 삭제 추가.
  • getLocalLogFile 메서드를 개선하여 현재 로그와 롤오버 로그를 일관성 있게 처리.
  • 임시 파일 생성 로직을 제거하여 코드 가독성과 유지보수성 향상.
  • 배포환경에서의 로깅 파일 디렉토리 새롭게 지정(로컬과 분리)

테스트 결과

권한문제로 elastic beanstalk에서 배포가 안됨. - HTTP 5xx 에러가 발생

/var/log 디렉토리에서 로그 디렉토리를 생성하려면 root 권한이 필요하다. Elastic Beanstalk 환경은 애플리케이션 사용자(webapp 등)로 실행되며, 디렉토리 생성 및 소유권 변경 작업이 실패할 수 있다고 한다.
(mkdir 또는 chown 작업이 실패하면 애플리케이션이 제대로 실행되지 않아 HTTP 5xx 에러가 발생할 수 있음)

일정

  • 추정 시간 : 3 day
  • 걸린 시간 : 1 day

@persi0815 persi0815 self-assigned this Jan 2, 2025
@persi0815 persi0815 linked an issue Jan 2, 2025 that may be closed by this pull request
@persi0815 persi0815 merged commit 889a4d8 into develop Jan 2, 2025
@persi0815 persi0815 changed the title [BUG FIX/90] S3 로그 업로드 로직 수정 및 병합 로직 제거 [BUG FIX/90] S3 로그 업로드 로직 수정 및 병합 로직 제거 & 로컬 디렉토리 변경 Jan 2, 2025
@persi0815 persi0815 changed the title [BUG FIX/90] S3 로그 업로드 로직 수정 및 병합 로직 제거 & 로컬 디렉토리 변경 [BUG FIX/90] S3 로그 업로드 로직 수정 및 병합 로직 제거 Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG FIX] 배포 환경에서의 AOP 버그 해결

1 participant