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

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

Merged
merged 9 commits into from
Dec 6, 2023

Conversation

yeseul106
Copy link
Member

@yeseul106 yeseul106 commented Dec 5, 2023

👩‍💻 Contents

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

📝 Review Note

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

📣 Related Issue

@yeseul106 yeseul106 added the 🎁 feature 새로운 기능 label Dec 5, 2023
@yeseul106 yeseul106 self-assigned this Dec 5, 2023
Copy link
Contributor

@rdd9223 rdd9223 left a comment

Choose a reason for hiding this comment

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

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

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

Choose a reason for hiding this comment

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

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

Copy link
Member Author

Choose a reason for hiding this comment

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

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

Copy link
Contributor

Choose a reason for hiding this comment

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

맞춰보시죠!!

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")
Copy link
Contributor

Choose a reason for hiding this comment

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

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

Copy link
Member Author

Choose a reason for hiding this comment

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

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

.collect(Collectors.toList());

if (userJoinedList.isEmpty()) {
throw new BaseException(HttpStatus.NO_CONTENT);
Copy link
Contributor

Choose a reason for hiding this comment

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

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

Copy link
Member Author

Choose a reason for hiding this comment

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

넵 ! 좋아요 !!

@rdd9223
Copy link
Contributor

rdd9223 commented Dec 5, 2023

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

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

@rdd9223
Copy link
Contributor

rdd9223 commented Dec 5, 2023

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

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

@yeseul106 yeseul106 closed this Dec 5, 2023
@yeseul106 yeseul106 reopened this Dec 5, 2023
@rdd9223
Copy link
Contributor

rdd9223 commented Dec 6, 2023

머지해도 조아용

@rdd9223 rdd9223 merged commit 43d7937 into develop Dec 6, 2023
@mikekks mikekks deleted the feature/#89 branch August 4, 2024 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎁 feature 새로운 기능 size/XXL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

내가 속한 모임 리스트 조회 API 구현
2 participants