Skip to content

Conversation

@joowojr
Copy link
Contributor

@joowojr joowojr commented Feb 4, 2025

📄 요약(Summary)

로그인 시도 감지 필터 로직 수정 및 로그인 실패 로깅 추가

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • 로그인 시도 감지 필터 로직 변경

    • session id로 로그인 사용자를 식별하던 방법에서, ip주소로 식별하도록 수정하였습니다.
    • 잠금된 계정에 한해서 30분이 지났을 경우 해당 로그 데이터를 삭제하도록 수정하였습니다.
  • 로그인 시도 감지 필터에서 계정이 잠김을 나타내는 auth_017을 반환하지 않고 auth_001을 반환하는 오류

    • try catch문으로 수정하여, 필터에서 authexception이 일어날 경우에 해당 에러를 catch해서 반환하도록 수정
  • 로그인 실패 로깅 추가

    • loggingaspect에서 로그인 성공 case와 실패 case를 분기처리하여, 실패 시에는 로그인 실패 데이터를 저장하도록 구현하였습니다.
  • 불필요한 response dto 삭제

    • 작업에 대해 update가 일어나는 것들에 대하여 불필요한 response dto를 반환하고 있어서 이를 모두 삭제하였습니다.

Requirements for Reviewer

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

PR Log

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

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

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

✅ 체크리스트(Checklist)

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

🚪 이슈 번호(Issue numbers)

Closes #256

@joowojr joowojr self-assigned this Feb 4, 2025
@joowojr joowojr added ✨ feature 구현·개선 사항에 관련된 내용입니다 HIGH 우선순위 상 labels Feb 4, 2025
@joowojr joowojr merged commit 47fc6c6 into develop Feb 4, 2025
1 check passed
@joowojr joowojr deleted the CLAP-241 branch February 4, 2025 09:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feature 구현·개선 사항에 관련된 내용입니다 HIGH 우선순위 상

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLAP-241 로그인 로깅 필터 로직 수정 및 감사 로그에 로그인 시도 횟수 추가

2 participants