7주차 과제 - Spring Security 인가(Authorization) 구현하기 #74

jdalma · 2022-09-19T06:59:33Z

TODO

사용자 정보는 자신의 것만 수정 가능하게
사용자 수정,삭제는 관리자만 가능하게
상품 정보는 자신의 것만 수정 가능하게
상품 수정,삭제는 관리자만 가능하게
userId에 따라서 권한을 따로 부여

로그인이 필요없는 API

로그인 - POST /session
회원 생성하기 - POST /users
고양이 장난감 목록 얻기 - GET /products
고양이 장난감 상세 조회하기 - GET /products/{id}

로그인이 필요한 API

고양이 장난감 등록하기 - POST /products
고양이 장난감 수정하기 - PATCH /products/{id}
고양이 장난감 삭제하기 - DELETE /products/{id}
회원 수정하기 - POST /users/{id}
회원 삭제하기 - DELETE /users/{id}

스프링 시큐리티를 처음 접해봐서 중요한 포인트나 키워드를 던져주신다면 열심히 조사해보겠습니다!
한 주간 잘 부탁드립니다!!

johngrib

오래간만입니다!

app/src/main/java/com/codesoom/assignment/config/SecurityJavaConfig.java

app/src/main/java/com/codesoom/assignment/controllers/ProductController.java

app/src/main/java/com/codesoom/assignment/filters/AuthenticationErrorFilter.java

app/src/test/java/com/codesoom/assignment/domain/UserTest.java

app/src/main/java/com/codesoom/assignment/dto/ProductData.java

app/src/test/java/com/codesoom/assignment/controllers/UserControllerTest.java

app/src/main/java/com/codesoom/assignment/controllers/ControllerErrorAdvice.java

app/src/main/java/com/codesoom/assignment/App.java

app/src/main/resources/application.yml

app/src/test/java/com/codesoom/assignment/controllers/UserControllerTest.java

app/src/main/java/com/codesoom/assignment/filters/JwtAuthenticationFilter.java

app/src/test/java/com/codesoom/assignment/controllers/ProductControllerTest.java

DTO -> Domain 메소드 추가

johngrib

잘 해나가고 계신 것 같습니다. 화이팅!

johngrib · 2022-09-22T12:24:33Z

app/src/main/java/com/codesoom/assignment/filters/JwtAuthenticationFilter.java

+import java.io.IOException;
+import java.util.Set;
+
+public class JwtAuthenticationFilter extends BasicAuthenticationFilter {


이 클래스의 용도와 기능을 JavaDoc으로 작성해 주세요.

d803878 이렇게 작성해 봤습니다 ㅎㅎ

현재 과제에서는 Bearer scheme과 JWT를 사용하고 있는데 BasicAuthenticationFilter클래스를 확장하고 있습니다

요약하면 이 필터는 인증 체계가 Basic 이고 Base64로 인코딩된 username:password 토큰이 있는 Authorization 의 HTTP 요청 헤더가 있는 모든 요청을 처리합니다.

BasicAuthenticationFilter클래스를 확장하기 보단 다른 필터를 찾아 확장하거나 구현하는게 맞는걸까요??
이해하기로는 Authorization scheme은 인증 체계를 알려주기만 하는거라고 이해했습니다 ㅎㅎ

설명이 필요 이상으로 자세한 것 같아요. 이 클래스의 "책임"만 간단하게 작성해 보세요.

johngrib · 2022-09-22T12:24:58Z

app/src/main/java/com/codesoom/assignment/security/UserAuthentication.java

+
+    private static List<GrantedAuthority> authorities() {
+        List<GrantedAuthority> authorities = new ArrayList<>();
+        // TODO : userId에 따라서 권한을 따로 부여


이 TODO는 언제 실행할 예정인가요? 아니면 이미 끝난 작업인가요?

어떻게 적용해야 할지 몰라서 스프링 문서 뒤져가면서 찾고 있었습니다!
계속 작업 중 입니다 !! 9e54f15

제가 생각하는 흐름은

userId -1을 관리자로 지정

User 엔티티에 enum role 필드 추가

사용자 생성 시 기본 사용자는 USER로 저장

JWT 페이로드에 userId와 role 정보를 같이 인코딩

사용자 또는 상품 도메인 접근 시 JWT에 담겨 있는 role정보를 확인해서 인가가 가능한지 구분

일단 이렇게 생각하고 3번까지 작업된 것 같습니다
시큐리티 설정에 문제가 있는지 401이 반환돼서 확인 작업 중에 있습니다!!

johngrib · 2022-09-22T12:26:25Z

app/src/test/java/com/codesoom/assignment/controllers/UserControllerTest.java

+        @Nested
+        @DisplayName("식별자에 해당하는 사용자가 존재하고 인증을 통과한다면")
+        class Context_ExistedIdAndValidToken{
+


사용자가 존재하는 상황을 Context_ExistedIdAndValidToken 에서 명시해 주세요. 이 컨텍스트가 표현하고자 하는 것을 이 컨텍스트에서 코딩하는 것입니다.

Describe_Context_It을 무의식적으로 작성한 것 같네요 ㅠㅠ
문맥별로 더 쪼개보니 반대 상황이 더 잘보이는 것 같습니다 ㅎㅎ
ef37c86

johngrib

테스트가 한결 보기 좋습니다. 그러고보니 벌써 금요일이군요.

johngrib · 2022-09-23T12:44:33Z

app/src/main/java/com/codesoom/assignment/filters/JwtAuthenticationFilter.java

+import java.io.IOException;
+import java.util.Set;
+
+public class JwtAuthenticationFilter extends BasicAuthenticationFilter {


설명이 필요 이상으로 자세한 것 같아요. 이 클래스의 "책임"만 간단하게 작성해 보세요.

jdalma · 2022-09-25T09:13:57Z

이번 주 과제는 조금 아쉽게 끝나는 것 같네요 ㅠ
스프링 시큐리티를 처음 사용해서 많이 헤맸네요 ㅎㅎ
그리고 종립님이 모킹을 안 쓰는 테스트 코드에 적응을 먼저 하는게 좋다 라고 말씀하신 게 이번에 와닿았습니다
격리된 테스트 코드를 짜는게 쉽지가 않네요 ㅠㅠ
그래도 더 공부해야할 키워드들도 많이 나왔고 많이 배웠습니다!!
한 주간 감사했습니다!!

회고

johngrib

한 주간 고생 많으셨습니다!

johngrib · 2022-09-25T12:00:47Z

app/src/main/java/com/codesoom/assignment/application/UserService.java

-    public User updateUser(Long id, UserModificationData modificationData) {
+    public User updateUser(Long id, UserModificationData modificationData , Long userId) throws AccessDeniedException {
+        if(!Objects.equals(id, userId)){
+            throw new AccessDeniedException("자원 접근이 거부되었습니다.");


만약 여러 나라의 언어로 서비스를 한다면 어떻게 대응하면 좋을지에 대해서 생각해 보세요.

johngrib · 2022-09-25T12:02:15Z

app/src/test/java/com/codesoom/assignment/controllers/UserControllerTest.java

+            class Context_ValidToken{
+
+                @Nested
+                @DisplayName("사용자 권한이라면")


관리자 권한이 아닌 경우로 표현할 수도 있겠습니다.

jdalma added 8 commits September 19, 2022 10:38

Spring Security 추가

3d1e47f

SpringSecurity Filter 추가로 인해 주석 처리

48a38f6

advice에서 InvalidTokenException 예외 주석 처리

1d029ac

TODO 추가

aeb3fb1

상품 삭제 시 토큰 없는 경우 테스트 코드 추가

6d7e5b8

SpringSecurity 인증 및 인가 추가

79e26d0

단일 import문으로 변경

7d0ffd6

User Password 암호화 , bcrypt 사용

4f9695f

johngrib reviewed Sep 19, 2022

View reviewed changes

jdalma added 3 commits September 20, 2022 17:33

Formatter 추가

2ae9974

authenticate() 테스트 코드 수정

f8f1731

User update() 테스트 코드 작성 중

b89ad93

johngrib reviewed Sep 20, 2022

View reviewed changes

jdalma added 3 commits September 21, 2022 17:18

주석 삭제

43ef954

try 수정

0cb6ec0

사용자 테스트 코드 수정

4587c85

johngrib reviewed Sep 21, 2022

View reviewed changes

jdalma added 6 commits September 22, 2022 12:45

DozerMapper 제거

9d7538d

테스트 코드 DisplayName 수정

05fc813

DozerMapper 제거

80d8717

DTO -> Domain 메소드 추가

ProductData toProduct() maker 필드 추가

f4614c9

AuthenticationFilter filterWithPathAndMethod() 수정

bf1ef94

ProductController 테스트 코드 content 부분 수정

15e299c

johngrib reviewed Sep 22, 2022

View reviewed changes

jdalma added 4 commits September 23, 2022 16:14

JwtAuthenticationFilter JavaDoc 작성

d803878

JwtUtil.encode() 수정

e5e9e6c

UserController create() 테스트 코드 추가

ef37c86

사용자별 권한 작업 중

9e54f15

johngrib approved these changes Sep 23, 2022

View reviewed changes

JwtAuthenticationFilter 주석 수정

3fd2645

jdalma added 5 commits September 25, 2022 14:05

user 자신의 정보만 수정 가능

0d5abc2

UserController 기존 테스트 통과하도록 수정

edc9fcf

jwt secret 수정

6c44e40

DTO 기본 생성자 추가

63ea12a

User 삭제는 관리자만 가능하게 수정

e69315f

jdalma force-pushed the main branch from dc1be85 to e69315f Compare September 25, 2022 07:57

jdalma added 2 commits September 25, 2022 16:59

ddl-auto: update 수정

c585ae8

과제 풀이 - 권한 적용

9079fd1

johngrib approved these changes Sep 25, 2022

View reviewed changes

johngrib merged commit b488d15 into CodeSoom:jdalma Sep 25, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

7주차 과제 - Spring Security 인가(Authorization) 구현하기 #74

7주차 과제 - Spring Security 인가(Authorization) 구현하기 #74

jdalma commented Sep 19, 2022 •

edited

johngrib left a comment

johngrib left a comment

johngrib Sep 22, 2022

jdalma Sep 23, 2022

johngrib Sep 23, 2022

johngrib Sep 22, 2022

jdalma Sep 22, 2022 •

edited

johngrib Sep 22, 2022

jdalma Sep 23, 2022

johngrib left a comment

johngrib Sep 23, 2022

jdalma commented Sep 25, 2022

johngrib left a comment

johngrib Sep 25, 2022

johngrib Sep 25, 2022

7주차 과제 - Spring Security 인가(Authorization) 구현하기 #74

7주차 과제 - Spring Security 인가(Authorization) 구현하기 #74

Conversation

jdalma commented Sep 19, 2022 • edited

TODO

로그인이 필요없는 API

로그인이 필요한 API

johngrib left a comment

Choose a reason for hiding this comment

johngrib left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jdalma Sep 22, 2022 • edited

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

johngrib left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jdalma commented Sep 25, 2022

johngrib left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

jdalma commented Sep 19, 2022 •

edited

jdalma Sep 22, 2022 •

edited