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

[NAYB-153] feat: api 권한 설정 #113

Merged
merged 3 commits into from
Sep 18, 2023
Merged

[NAYB-153] feat: api 권한 설정 #113

merged 3 commits into from
Sep 18, 2023

Conversation

hseong3243
Copy link
Collaborator

@hseong3243 hseong3243 commented Sep 18, 2023

⛏ 작업 사항

  • api 별 권한 설정을 추가했습니다. 추후 새로운 api가 추가되는 경우 알려주세요.
  • @WebMvcTest 설정이 일부 변경되었습니다. 기존에 셋업 메서드를 통해서 인증 객체를 미리 설정하는 것에서 JWT필터를 적용하여 직접 accessToken을 헤더에 추가하는 방식으로 변경하였습니다.
    • 기존 mockMvc 테스트에는 모두 적용시켜두었습니다. 새롭게 컨트롤러 테스트를 수행할 때, @LoginUser를 사용한다면 mockMvc.perform(~).header(AUTHORIZATION, accessToken)을 추가해주세요. BaseControllerTest에 필드에 AuthorizationaccessToken을 추가해두었기 때문에 앞에 작성한 코드처럼 작성하시면 됩니다.
  • 회원 연동 해제 로직에서 데이터베이스의 oauth2 데이터가 삭제되지 않는 문제를 발견하여 로직 일부를 수정하였습니다.

📝 작업 요약

  • BaseControllerTest 인증 객체 설정 제거, JwtAuthenticationFilter 적용 및 자식 테스트 클래스에 반영
  • RestTemplateOAuthClient 회원 연동 해제 시 oauth2 데이터 삭제 메서드 호출 추가
  • WebSecurityConfig 유저 role에 따라 api 권한 검증 추가

💡 관련 이슈

@github-actions
Copy link

Test Results

352 tests  ±0   352 ✔️ ±0   9s ⏱️ -1s
173 suites ±0       0 💤 ±0 
173 files   ±0       0 ±0 

Results for commit a164159. ± Comparison against base commit b52b2e4.

Copy link
Member

@bjo6300 bjo6300 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~!

@hseong3243 hseong3243 merged commit c352079 into develop Sep 18, 2023
3 checks passed
@hseong3243 hseong3243 deleted the feature/NAYB-153 branch September 18, 2023 06:56
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