Skip to content

[#91]feat: securityConfigTest 수정#122

Merged
soyeonkim8888 merged 1 commit into
IssueDiver:devfrom
soyeonkim8888:feature/91-logout
Sep 10, 2025
Merged

[#91]feat: securityConfigTest 수정#122
soyeonkim8888 merged 1 commit into
IssueDiver:devfrom
soyeonkim8888:feature/91-logout

Conversation

@soyeonkim8888
Copy link
Copy Markdown
Contributor

연관된 이슈

#91


작업 내용

securityConfigTest.java 파일 수

4가지 항목에서 지속적 오류 발생하여 수정
SecurityConfigTest > HTTP 메서드별 접근 제어 - DELETE
SecurityConfigTest > 유효한 JWT 토큰으로 /auth/logout 접근 허용
SecurityConfigTest > JWT 필터 체인 순서 확인
SecurityConfigTest > 유효한 JWT 토큰으로 /auth/users/{id} 접근 허용

📸 스크린샷 (선택)

PR 유형

어떤 변경 사항이 있나요?

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

리뷰 요구사항(선택)

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

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?


PR Checklist

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

  • 커밋 메시지 컨벤션에 맞게 작성했습니다.
  • 변경 사항에 대한 테스트를 했습니다.(버그 수정/기능에 대한 테스트).
  • main branch가 아닌 dev branch에 PR 요청을 했습니다. (main branch에 바로 PR&merge하지 않기).

@soyeonkim8888 soyeonkim8888 merged commit 35262cf into IssueDiver:dev Sep 10, 2025
soyeonkim8888 added a commit that referenced this pull request Sep 11, 2025
[91] feat: #122 상태로 되돌리기 위해 다시 커밋
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants