Skip to content

Conversation

@parkjaehak
Copy link
Collaborator

@parkjaehak parkjaehak commented Feb 2, 2025

📄 요약(Summary)

로그인 로그 기록, 서비스 로그 기록, 로그 조회

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • HttpServletRequest 캐싱은 filter단에서 처리하여 aop 레이어에서 찍히지 않는 문제 해결
  • LogStatus에 대한 구분을 세분화하여 애노테이션을 통해 로그를 찍을 컨트롤러 메서드 위에 정의하면 해당하는 로그 기록을 찍을 수 있도록 설정
  • 서비스 정의 예외의 경우 BaseException을 발생시킬때 사용하는 BaseErrorCode에 정의된 상태코드와 값을 이용
  • 나머지 런타임, 예상치 못한 에러의 경우 handlerExceptionResolver에서 파싱한 ModelAndView에 정의된 상태코드를 이용

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

  • getFailedAttemptCount의 경우 아직 구현하지 못해서 구현예정입니다.

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • PR 양식에 맞게 작성했습니다
  • 모든 테스트가 통과했습니다
  • 프로그램이 정상적으로 작동합니다
  • 적절한 PR 라벨을 설정했습니다
  • 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #195

@parkjaehak parkjaehak added ✨ feature 구현·개선 사항에 관련된 내용입니다 ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다. HIGH 우선순위 상 labels Feb 2, 2025
@parkjaehak parkjaehak requested a review from Sihun23 February 2, 2025 07:57
@parkjaehak parkjaehak self-assigned this Feb 2, 2025
@joowojr
Copy link
Contributor

joowojr commented Feb 2, 2025

@parkjaehak 로그인 시도 실패 기록의 경우 구현중에 있어서 나중에 pr올리도록 하겠습니다!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

두 파일을 합쳐 주셔도 될것 같습니다!

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

String categoryName,
String title,
String processorName,
String processorUrl,
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

dto에 url 필드를 추가하신 이유가 있을까요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

작업 목록 조회시 프로필 이미지가 필요할 것 같습니다. 피그마 상에도 존재하고 장호님께서 요청을 주셨습니다.

@parkjaehak parkjaehak merged commit 7b0ef5a into develop Feb 2, 2025
1 check passed
@parkjaehak parkjaehak deleted the CLAP-196 branch February 3, 2025 05:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feature 구현·개선 사항에 관련된 내용입니다 HIGH 우선순위 상 ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-196 로그인 로그 리팩토링

4 participants