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

86 모든 requesturl 허용가능하도록 수정 #87

Merged
merged 1 commit into from
Sep 26, 2023

Conversation

handseokjin
Copy link
Contributor

  • 🔀 PR 제목의 형식을 잘 작성했나요? e.g. [feat] PR을 등록한다.
  • 💯 테스트는 잘 통과했나요?
  • 🏗️ 빌드는 성공했나요?
  • 🧹 불필요한 코드는 제거했나요?
  • 💭 이슈는 등록했나요?
  • 🏷️ 라벨은 등록했나요?
  • 💻 git rebase를 사용했나요?

작업 내용

기존에 제 부분만 테스트한다고 api/v1/auth로 시작하는 url만 허용되도록 했는데 다른 url도 모두 허용되도록 임시로 변경해 두겠습니다. 추후에 다시 정리하도록 하겠습니다!

위치 : SecurityConfig.java
변경사항 : anyRequest().authenticated()를 anyRequest().permitAll()로 임시 변경하였습니다.

스크린샷

주의사항

Closes #{86}

기존에 제 부분만 테스트한다고 api/v1/auth로 시작하는 url만 허용되도록 했는데 다른 url도 모두 허용되도록 임시로 변경해 두겠습니다.
추후에 다시 정리하도록 하겠습니다!
@handseokjin handseokjin linked an issue Sep 26, 2023 that may be closed by this pull request
@handseokjin handseokjin changed the base branch from main to be September 26, 2023 08:37
@handseokjin handseokjin added 💙 BE 백엔드 📦 chore 의존성, 라이브러리, 잡다한 일 🐞 fix 버그 수정 labels Sep 26, 2023
@handseokjin handseokjin self-assigned this Sep 26, 2023
@acisliver acisliver merged commit 2b6f6f7 into be Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💙 BE 백엔드 📦 chore 의존성, 라이브러리, 잡다한 일 🐞 fix 버그 수정
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

모든 requestUrl 허용가능하도록 수정
2 participants