Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix/exception logger #191

Open
wants to merge 17 commits into
base: main
Choose a base branch
from
Open

Fix/exception logger #191

wants to merge 17 commits into from

Conversation

OKSUchun
Copy link
Contributor

@OKSUchun OKSUchun commented Nov 8, 2024

PR 타입(하나 이상의 PR 타입을 선택해주세요)

-[] 기능 추가
-[] 기능 삭제
-[] 버그 수정
-[v] 의존성, 환경 변수, 빌드 관련 코드 업데이트

반영 브랜치

fix/exception-logger -> dev

변경 사항

GlobalExceptionHandler

  • 설명:
    • CustomException, IllegalArgumentException, NullPointerException에 대한 특정 Exception 별 핸들러를 사용했습니다.

CustomException

  • 운영시 debugging 용이하도록 로깅 별도로 설정
  • Exception 발생시 parameter logging 하도록 설정
  • logMessage 라는 새로운 필드를 생성

테스트 결과

  • 정상적으로 동작합니다!

OKSUchun and others added 16 commits October 14, 2024 16:17
* [FIX] add SSL configuration

* [FIX] change backend port on docker-compose.yml

* [FIX] add 80 backend port on docker-compose.yml
* [FIX] add SSL configuration

* [FIX] change backend port on docker-compose.yml

* [FIX] add 80 backend port on docker-compose.yml

* [FIX] configure server.port=8080
* [FIX] add SSL configuration

* [FIX] change backend port on docker-compose.yml

* [FIX] add 80 backend port on docker-compose.yml

* [FIX] configure server.port=8080

* [FIX] Add volumes to access keys

* [FIX] change port 8443

* [FIX] change port 8080

* [FIX] change spring boot port to 8080
* [FIX] add SSL configuration

* [FIX] change backend port on docker-compose.yml

* [FIX] add 80 backend port on docker-compose.yml

* [FIX] configure server.port=8080

* [FIX] Add volumes to access keys

* [FIX] change port 8443

* [FIX] change port 8080

* [FIX] change spring boot port to 8080

* [FIX] change spring boot port to 8443
* [FIX] add SSL configuration

* [FIX] change backend port on docker-compose.yml

* [FIX] add 80 backend port on docker-compose.yml

* [FIX] configure server.port=8080

* [FIX] Add volumes to access keys

* [FIX] change port 8443

* [FIX] change port 8080

* [FIX] change spring boot port to 8080

* [FIX] change spring boot port to 8443

* [FIX] change spring boot port to 8080
- env 경로 설정
- port 443 설정
- dev, local : jpa, redis, cors 관련 logging level 설정 제거
- 로그 가독성 향상 목적
- spring.jpa.properties.hibernate.dialect 제거
- spring.jpa.open-in-view=true 기재
- 목적: DTO 동등성 비교를 위한 EqualsAndHashCode 설정
- lombok EqualsAndHashCode annotation 추가
- Exception 발생 시 log level, logging 양식 설정
- CustomException : field logMessage 생성
- CustomException 발생시 개발자가 디버그에 용이하도록 logMessage 전달
@OKSUchun OKSUchun requested a review from yuha00e November 8, 2024 09:03
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