Modify/#66 modify authority #67

dong2ast · 2023-07-25T07:28:26Z

📌 관련 이슈

closed #66
URL이 많이 바뀌어서 아직은 merge 안하는게 좋을 것 같습니다.

✨ 과제 내용

Authority Enum 필드 추가
Boolean 필드 지우고 Enum type으로 확인하게 변경
작가 관련된 URI 한 컨트롤러에서 관리하게 변경
Security Config에 작가 URI에 대한 접근 권한 설정
token으로 유저 정보 받아오게 변경 -> Path Variable 사용 x

📸 스크린샷(선택)

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

…한 구별하게 변경

onpyeong · 2023-07-26T11:41:03Z

src/main/java/org/sophy/sophy/controller/AuthController.java

+        /**
+         * HttpServletRequest나 HttpServletResponse 객체가 Service 계층으로 넘어가는 것은 좋지 않다.
+         * request, response는 컨트롤러 계층에서 사용되는 객체이며, Service 계층이 request와 response를 알 필요가 없다.
+         */
+        String accessToken = tokenProvider.resolveAccessToken(request);


오 배워갑니다!👍

onpyeong · 2023-07-26T12:28:09Z

src/main/java/org/sophy/sophy/controller/HomeController.java

+    @GetMapping("/myhome") //회원 홈 조회 (작가와 주민 구분은 서비스 단 내에서)
+    public ApiResponseDto<HomeResponseDto> getHome(@AuthenticationPrincipal User user) {
+        Long memberId = memberRepository.getMemberByEmail(user.getUsername()).getId();


controller에서 repository를 호출하는 것이 아키텍처 관점이나 유지 보수 측면에서 좋지 않은 것 같아서
email 정보를 service로 넘겨서 service 내에서 repository를 호출해서 해당 이메일의 유저가 있는지를 검증하도록 하는게 어떨까요?

onpyeong

스프링 시큐리티 role 적용하다니 굿굿!👍
근데 언제 쉽니까....

dong2ast added 7 commits July 25, 2023 10:04

add: enum 권한들 추가

30547cd

refactor: member 엔티티에 isAuthor, isOperator 필드를 없애고 Authority 필드를 통해 권…

b04ed76

…한 구별하게 변경

refactor: token 정보로 User 정보 받아오는 것으로 수정 | 기존 URI도 memberId 직접 다루지 않게 수정

134b450

refactor: 작가 관련 메서드 다루는 컨트롤러 통일 및 권한=작가 외에는 접근 금지 설정

59668bc

fix: 북토크가 완료되었을 때 작가의 소피스토리도 업데이트 되게끔 변경

46f473f

add: 헤더에 있는 access토큰과 refresh토큰을 받아오는 메서드 추가

995ce4d

modify: 로그아웃시 헤더에서 access토큰만 받아오게 변경

bc2b0ae

dong2ast added the 개발 새로운 기능을 추가 했을 때 label Jul 25, 2023

dong2ast self-assigned this Jul 25, 2023

dong2ast linked an issue Jul 25, 2023 that may be closed by this pull request

[modify] authority를 이용해 권한 분리 #66

Closed

5 tasks

onpyeong reviewed Jul 26, 2023

View reviewed changes

onpyeong approved these changes Jul 26, 2023

View reviewed changes

dong2ast merged commit 37c673e into main Jul 27, 2023
1 check passed

dong2ast deleted the modify/#66-modify-authority branch July 27, 2023 04:57

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Modify/#66 modify authority #67

Modify/#66 modify authority #67

dong2ast commented Jul 25, 2023

onpyeong Jul 26, 2023

onpyeong Jul 26, 2023

onpyeong left a comment

Modify/#66 modify authority #67

Modify/#66 modify authority #67

Conversation

dong2ast commented Jul 25, 2023

📌 관련 이슈

✨ 과제 내용

📸 스크린샷(선택)

📚 레퍼런스 (또는 새로 알게 된 내용) 혹은 궁금한 사항들

onpyeong Jul 26, 2023

Choose a reason for hiding this comment

onpyeong Jul 26, 2023

Choose a reason for hiding this comment

onpyeong left a comment

Choose a reason for hiding this comment