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

[BE] chore: 존재하지 않는 경로로 요청 왔을 때, 예외 메시지만 로깅 #730

Merged
merged 1 commit into from
Nov 4, 2024

Conversation

takoyakimchi
Copy link
Contributor

이슈

개발 사항

  • 존재하지 않는 경로로 요청이 들어오면 stack trace를 모두 로깅하고 있어서 로그 확인이 어려움
  • 예외 메시지만 로깅해서 어떤 경로로 공격을 시도했는지만 파악하도록 수정

@takoyakimchi takoyakimchi added the 🖥 backend backend label Nov 4, 2024
@takoyakimchi takoyakimchi added this to the Sprint7 milestone Nov 4, 2024
@takoyakimchi takoyakimchi self-assigned this Nov 4, 2024
Copy link

github-actions bot commented Nov 4, 2024

Test Results

223 tests  +163   223 ✅ +163   19s ⏱️ +18s
 46 suites + 37     0 💤 ±  0 
 46 files   + 37     0 ❌ ±  0 

Results for commit dc9db25. ± Comparison against base commit 262495f.

This pull request removes 60 and adds 223 tests. Note that renamed tests count towards both.
com.happy.friendogly.presentation.ui.club.detail.ClubDetailViewModelTest ‑ 모임 상세 로드에 실패하면 사용자에게 실패를 알리는 이벤트를 발생시킨다()
com.happy.friendogly.presentation.ui.club.detail.ClubDetailViewModelTest ‑ 모임 상세가 로드된다()
com.happy.friendogly.presentation.ui.club.detail.ClubDetailViewModelTest ‑ 모임 참여에 성공하면 사용자에게 참여 성공을 알리는 이벤트가 발생된다()
com.happy.friendogly.presentation.ui.club.detail.ClubDetailViewModelTest ‑ 모임 참여할 수 없는 경우 사용자에게 참여 실패를 알리는 이벤트가 발생한다()
com.happy.friendogly.presentation.ui.club.list.ClubListViewModelTest ‑ 등록한 주소가 없는 경우 사용자에게 주소 등록을 요구하는 화면을 표시한다()
com.happy.friendogly.presentation.ui.club.list.ClubListViewModelTest ‑ 등록한 주소의 모임 리스트가 로드된다()
com.happy.friendogly.presentation.ui.mypage.MyPageViewModelTest ‑ 내가 방장인 모임 이동 이벤트가 발생한다()
com.happy.friendogly.presentation.ui.mypage.MyPageViewModelTest ‑ 내가 참여한 모임 이동 이벤트가 발생한다()
com.happy.friendogly.presentation.ui.mypage.MyPageViewModelTest ‑ 사용자 프로필 편집 이동 이벤트가 발생한다()
com.happy.friendogly.presentation.ui.mypage.MyPageViewModelTest ‑ 사용자와 펫 정보가 로드된다()
…
com.happy.friendogly.DataSourceRoutingTest ‑ 읽기 전용 트랜잭션이 아니면, Writer DB 데이터소스가 바운딩된다.
com.happy.friendogly.DataSourceRoutingTest ‑ 읽기전용 트랜잭션이면 reader DB 데이터소스가 바운딩된다.
com.happy.friendogly.chat.controller.ChatMessageControllerTest ‑ 두 시점 사이의 채팅 메시지를 조회할 수 있다.
com.happy.friendogly.chat.service.ChatCommandServiceTest ‑ 채팅 메시지를 DB에 저장한다.
com.happy.friendogly.chat.service.ChatCommandServiceTest ‑ 채팅 입장 메시지를 DB에 저장한다.
com.happy.friendogly.chat.service.ChatCommandServiceTest ‑ 채팅 퇴장 메시지를 DB에 저장한다.
com.happy.friendogly.chat.service.ChatQueryServiceTest ‑ since 시간과 until 시간 사이의 (포함 X) 채팅 내역을 조회할 수 있다.
com.happy.friendogly.chat.service.ChatQueryServiceTest ‑ 채팅 메시지 시간 범위 조회에서, 시간 범위가 잘못된 경우 예외가 발생한다.
com.happy.friendogly.chat.service.ChatQueryServiceTest ‑ 채팅방에 들어가 있지 않은 Member는 채팅 내역을 조회할 수 없다.
com.happy.friendogly.chat.service.ChatRoomCommandServiceTest ‑ 1대1 채팅방 저장 과정에서, 이미 채팅방이 존재하면 기존 채팅방 ID를 반환한다.
…

Copy link
Member

@jimi567 jimi567 left a comment

Choose a reason for hiding this comment

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

로그가 너무 많이 쌓여서 그른가용?

@takoyakimchi
Copy link
Contributor Author

@jimi567 넵 이것 때문에 로그 확인이 넘 어렵네요 😭

Copy link
Contributor

@J-I-H-O J-I-H-O left a comment

Choose a reason for hiding this comment

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

존재하지 않는 경로는 항상 같은 예외일테니 굳이 스택 트레이스를 모두 찍지 않아도 될 것 같아요!
합리적입니다~~ 👍

@takoyakimchi takoyakimchi merged commit a2e29e1 into develop Nov 4, 2024
3 checks passed
@takoyakimchi takoyakimchi deleted the feature/issue-727 branch November 5, 2024 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

존재하지 않는 경로로 요청이 들어왔을 때 로깅 정보를 최소화
3 participants