refactor: 비정상적인 서버 접근 시 슬랙 알림 전송 로직 수정 완료 #583
Merged
+58
−26
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
비정상적인 접근 패턴 탐지와 슬랙 알림 전송 관련 코드를 리팩토링하였습니다. 이번 리팩토링을 통해 불필요한 슬랙 알림을 줄이고, 최초 비정상적인 접근 시에만 알림이 전송되도록 변경하였습니다. 또한, 코드의 가독성과 유지보수성을 높였습니다.
Tasks
슬랙 알림 전송 로직 변경
비정상적인 접근 패턴 탐지 코드 분리
SecurityPatternChecker
클래스를 통해 의심스러운 경로 패턴 탐지를 담당하도록 코드 분리.InvalidEndpointAccessFilter
리팩토링handleSuspiciousAccess
,logSuspiciousAccess
등으로 메소드를 분리하여 단일 책임 원칙을 준수하도록 개선.ETC