(테스트용 DTO와 Controller 분리)Log Filter 구현 완료 - Githubissues

have-a-meal / have-a-meal-server

1 stars 0 forks source link

(테스트용 DTO와 Controller 분리)Log Filter 구현 완료 #36

Closed wjdwnsdnjs13 closed 6 months ago

wjdwnsdnjs13 commented 6 months ago

개요

이전 풀리퀘 시 조언 받은대로 Log Filter 테스트용 DTO와 Controller를 분리했습니다!
Http 요청에 대한 정보를 확인 할 수 있는 Log Filter를 구현했습니다.
인터셉터의 경우 시큐리티 필터를 통과하지 못 한 요청에 대해서는 확인할 수 없는 문제가 있기에 필터로 사용했습니다.
Response에 대해서는 아직 구현하지 않았습니다. 필요한 데이터나 정보들이 결정나면 추가로 구현하도록 하겠습니다.
PUT, DELETE 요청에 대해서는 로깅이 되는지 여부까지만 확인했습니다. 추후 수정 사항있을 경우 추가로 구현하겠습니다!

Pull_Request 유형

어떤 변경 사항이 있나요?

[x] 새로운 기능 추가
[ ] 버그 수정
[ ] CSS 등 사용자 UI 디자인 변경
[ ] 코드에 영향을 주지 않는 변경사항(오타 수정, 탭 사이즈 변경, 변수명 변경)
[ ] 코드 리팩토링
[ ] 주석 추가 및 수정
[ ] 문서 수정
[x] 테스트 추가, 테스트 리팩토링
[ ] 빌드 부분 혹은 패키지 매니저 수정
[ ] 파일 혹은 폴더명 수정
[ ] 파일 혹은 폴더 삭제

Pull_Request Checklist

다음 요구 사항을 충족하는지 확인하세요.

[x] 커밋 메시지 컨벤션에 맞게 작성했습니다. Wiki 커밋 컨벤션 참고
[x] 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).

jaewoong-gwon commented 6 months ago

사실 빈 자동 등록 범위에 TEST 패키지가 왜 포함되서 정상 동작하는지 모르겠네요 ㅎㅎ; 찾아 와야할듯 합니다