[#89] feat: 내가 속한 모임 리스트 조회 API 구현 #93

yeseul106 · 2023-12-05T08:53:01Z

👩‍💻 Contents

저희가 사용하고 있는 Spring boot가 3.X 버전임에 따라 hibernate 역시 버전 6을 권장하고 있어서 해당 버전에 맞게 PhysicalNamingStrategy로 잘 변경되도록 수정하였습니다.
내가 개설한 모임과 내가 들어간 모임 리스트를 합쳐서 조회할 수 있도록 API를 구현했습니다.
일단, 기존 Nest.js에서 발급한 토큰으로 Spring 서버에서도 동일하게 통신할 수 있도록 Spring Security를 붙여 구현하였습니다.
Swagger 세팅도 간단히 해두었습니다 !

📝 Review Note

Swagger 세팅 시, 기존 nest.js에서 Swagger 관련 작업 #43 에 맞게 세팅해두신 것을 확인은 했는데 맥락 파악이 부족해서 일단은 기본으로 세팅해두었습니다 ! 해당 부분 여쭤보고 수정 필요할 시, 수정 해놓겠습니다 !
현재 저희 JWT secret key가 Spring Security에서 JWT의 서명 검증에 사용되는 키의 크기가 설정한 알고리즘에 대해 충분하지 않아 에러가 납니다 ㅠㅠ (충분히 길게 수정하고 테스트 해보면 위와 같이 오류 없이 잘 돌아갑니다 !) 따라서 nest.js의 키값과 spring의 키값을 충분한 크기로 변경하고자 하는데 어떠신지 의견 여쭙니다 !

📣 Related Issue

closed 내가 속한 모임 리스트 조회 API 구현 #89

…이 되도록 수정

rdd9223

너무너무너무너무 고생했습니다!!

rdd9223 · 2023-12-05T13:37:47Z

main/src/main/java/org/sopt/makers/crew/main/common/config/jwt/JwtAuthenticationFilter.java

+      if (jwtTokenProvider.validateToken(accessToken)
+          == VALID_JWT_TOKEN) {     // 토큰이 존재하고 유효한 토큰일 때만
+        Integer userId = jwtTokenProvider.getAccessTokenPayload(accessToken);
+        UserAuthentication authentication = new UserAuthentication(userId, null,


굉장히.. 특이한... 줄바꿈 형식이네요?

아니 이거 무슨 구글 자바 스타일 가이드 컨벤션으로 맞춰놓은건데 ,,, 다른 걸로 맞출까요 ,,,? 허허

맞춰보시죠!!

rdd9223 · 2023-12-05T13:42:42Z

main/src/main/java/org/sopt/makers/crew/main/entity/user/User.java

 import org.sopt.makers.crew.main.entity.user.vo.UserActivityVO;

 @Entity
 @Getter
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
-@Table(name = "user")
+@Table(name = "user", schema = "web_dev")


요렇게 스키마 지정해두면 prod환경에서는 자동적으로 변경되는 건가요?? (몰라서 물음)

아뇨 ㅠ 이거 자꾸 테이블을 못찾아서 테스트용으로 붙여놨었는데 PhysicalNamingStrategy 구현체 직접 갈아껴서 해결해가지고,,! 요 부분 삭제 해두겠습니다 !

rdd9223 · 2023-12-05T13:45:28Z

main/src/main/java/org/sopt/makers/crew/main/user/v2/service/UserV2ServiceImpl.java

+        .collect(Collectors.toList());
+
+    if (userJoinedList.isEmpty()) {
+      throw new BaseException(HttpStatus.NO_CONTENT);


exception도 나중에 같이 한번 정의해봅시다!!

넵 ! 좋아요 !!

rdd9223 · 2023-12-05T13:47:56Z

Swagger 세팅 시, 기존 nest.js에서 #43 에 맞게 세팅해두신 것을 확인은 했는데 맥락 파악이 부족해서 일단은 기본으로 세팅해두었습니다 ! 해당 부분 여쭤보고 수정 필요할 시, 수정 해놓겠습니다 !

지금처럼 해두어도 괜찮을 것 같아요! 이원화 된 동안은 어쩔 수 없을 듯!

rdd9223 · 2023-12-05T13:49:05Z

현재 저희 JWT secret key가 Spring Security에서 JWT의 서명 검증에 사용되는 키의 크기가 설정한 알고리즘에 대해 충분하지 않아 에러가 납니다 ㅠㅠ (충분히 길게 수정하고 테스트 해보면 위와 같이 오류 없이 잘 돌아갑니다 !) 따라서 nest.js의 키값과 spring의 키값을 충분한 크기로 변경하고자 하는데 어떠신지 의견 여쭙니다 !

바꾸시죠!! 저도 이거 언제까지 짧은 걸로 할 수는 없다고 생각은 했어서 길게 한번 바꾸면 될 것 같아요. 기존 사용자들은 한번 더 로그인 해야겠지만.... 네이밍은 센스있게 한번 가져가보죠! 이거 플그랑 맞춰도 괜찮을지도?

rdd9223 · 2023-12-06T14:05:48Z

머지해도 조아용

yeseul106 added 8 commits December 2, 2023 20:51

[#89] chore: 불필요한 import문 삭제

2c2ce90

[#89] config: hibernate-types 사용을 위해 의존성 추가

2ca99cf

[#89] fix: hibernate 6에 맞춰서 PhysicalNamingStrategyStandardImpl 네이밍 전략…

7a2a611

…이 되도록 수정

[#89] rename: 공통 응답 모델 클래스명 수정

06382b3

[#89] feat: 내가 속한 모임 리스트 조회 api 구현

e40fd31

[#89] add: Spring Security 세팅

1f1cfa1

[#89] add: Swagger 관련 config 추가

95feae0

[#89] fix: security를 붙임에 따라 jwt 토큰 유저의 데이터를 내려주도록 수정

7afb7b5

yeseul106 added the 🎁 feature 새로운 기능 label Dec 5, 2023

yeseul106 requested a review from rdd9223 December 5, 2023 08:53

yeseul106 self-assigned this Dec 5, 2023

pull-request-size bot added the size/XXL label Dec 5, 2023

rdd9223 approved these changes Dec 5, 2023

View reviewed changes

yeseul106 closed this Dec 5, 2023

yeseul106 reopened this Dec 5, 2023

[#89] fix: 불필요한 코드 제거 및 jwt 관련 세팅 정리

df3756e

rdd9223 merged commit 43d7937 into develop Dec 6, 2023

mikekks deleted the feature/#89 branch August 4, 2024 08:30

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[#89] feat: 내가 속한 모임 리스트 조회 API 구현 #93

[#89] feat: 내가 속한 모임 리스트 조회 API 구현 #93

yeseul106 commented Dec 5, 2023 •

edited

Loading

rdd9223 left a comment

rdd9223 Dec 5, 2023

yeseul106 Dec 5, 2023

rdd9223 Dec 6, 2023

rdd9223 Dec 5, 2023

yeseul106 Dec 5, 2023

rdd9223 Dec 5, 2023

yeseul106 Dec 5, 2023

rdd9223 commented Dec 5, 2023 •

edited

Loading

rdd9223 commented Dec 5, 2023

rdd9223 commented Dec 6, 2023

[#89] feat: 내가 속한 모임 리스트 조회 API 구현 #93

[#89] feat: 내가 속한 모임 리스트 조회 API 구현 #93

Conversation

yeseul106 commented Dec 5, 2023 • edited Loading

👩‍💻 Contents

📝 Review Note

📣 Related Issue

rdd9223 left a comment

Choose a reason for hiding this comment

rdd9223 Dec 5, 2023

Choose a reason for hiding this comment

yeseul106 Dec 5, 2023

Choose a reason for hiding this comment

rdd9223 Dec 6, 2023

Choose a reason for hiding this comment

rdd9223 Dec 5, 2023

Choose a reason for hiding this comment

yeseul106 Dec 5, 2023

Choose a reason for hiding this comment

rdd9223 Dec 5, 2023

Choose a reason for hiding this comment

yeseul106 Dec 5, 2023

Choose a reason for hiding this comment

rdd9223 commented Dec 5, 2023 • edited Loading

rdd9223 commented Dec 5, 2023

rdd9223 commented Dec 6, 2023

yeseul106 commented Dec 5, 2023 •

edited

Loading

rdd9223 commented Dec 5, 2023 •

edited

Loading