코드 리뷰를 위한 임시 PR

[KimGyeongHo-com]

코드 리뷰를 위한 임시 PR

[jinyngg]

@KimGyeongHo-com

안녕하세요. 경호님 코드리뷰를 맡은 4조 장진영입니다.
경호님이랑은 그룹 스터디를 꾸준히 같이 해왔어서 항상 열심히 하시는 분인 것도 알고, 서버 관련해서 도움도 많이 주셨어서 배우는 입장에서 리뷰 진행할게요!

public ApiDataResponse<SignupDto.ResponseSignupSuccess> join(SignupDto joinRequestDto){
...

회원 가입이 진행될 때 데이터가 저장됨으로 @transaction 어노테이션이 사용되어야 할 것 같습니다.

member.setId(1l); 이 부분은 아이디가 고정적으로 들어가는 것 같습니다. 테스트를 위해 작성해 놓은 부분인지 궁금합니다!

MemberRole.ROLE_USER 를 Set하고 있는데 이번에 공부하면서 배운 부분인데 @Build.default를 Entity에 지정해두면 default 값을 User로 설정이 가능할 것 같습니다.

UsernamePasswordAuthenticationToken 로그인 과정에서 사용하지 않는 객체가 정의되어있는 것 같습니다!

이전에 JWT를 구현했을 때 JWT도 JwtAccessDeniedHandler.java 과 같은 핸들러를 구현할 수 있다는걸 몰랐는데 배웠습니다. 감사합니다!