feat: oauth2 로그인 by west-eastH · Pull Request #8 · Runners-Fight/Backend

west-eastH · 2025-06-29T13:15:22Z

✨ 연관된 이슈

close #7

📝 작업 내용 (주요 변경 사항)

멤버 엔티티 정의
JWT 관련 클래스 추가
OAuth2 관련 인터페이스 & 클래스 추가
OAuth2 로그인 구현

💬 리뷰 요구사항

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

createdAt, updatedAt, deletedAt 하나로 묶어서 사용하는거 알려주세요,,
SignupResponse에서 팩토리 메서드 패턴 사용하긴 했는데 null null null null이 들어가서 더 좋은 방법 있으면 코멘트 남겨주세요!

저희 테스트 커버리지가 너무 높은 것 같은듯,, 서지님 한번 해보시고 같이 조정해봐요~

github-actions · 2025-06-29T13:16:25Z

Test Coverage Report

Overall Project	46.31% `-36.19%`	❌
Files changed	55.58%	❌

File	Coverage
GoogleUserInfo.java	100%	🍏
SignupRequest.java	100%	🍏
SignupResponse.java	100%	🍏
TokenResponse.java	100%	🍏
Role.java	100%	🍏
OAuthType.java	100%	🍏
Gender.java	100%	🍏
ExceptionCode.java	100%	🍏
Member.java	100%	🍏
AuthService.java	98% `-2%`	🍏
CustomUserDetails.java	46.34% `-53.66%`	❌
JwtTokenProvider.java	1.92% `-98.08%`	❌
OAuth2UserInfoFactory.java	0%	❌
SecurityConfig.java	0%	❌
JwtAuthenticationFilter.java	0%	❌
CodeRequest.java	0%	❌
AuthController.java	0%	❌

choiseoji

수고하셨습니다!!
너무 너무 깔끔한 코드여서 나중에 많이 참고하겠습니다 ~ 😃

choiseoji · 2025-06-29T13:48:26Z

Makefile

+.PHONY: all build down re
+
+all: build copy up
+
+build:
+	@./gradlew clean build
+
+copy:
+	@cp ./build/libs/backend-0.0.1-SNAPSHOT.jar ./app.jar
+
+up:
+	@docker compose up --build -d
+
+down:
+	@docker compose down
+
+re: down up


혹시 makefile 어디에 사용되는건가요??

이거 빌드하고 docker compose 돌리는거 귀찮아서 만들었어요😅

choiseoji · 2025-06-29T13:55:27Z

src/main/java/run/backend/domain/auth/dto/response/TokenResponse.java

+public record TokenResponse(String accessToken, String refreshToken) {
+
+}


토큰 헤더에 안 넣고 body에 담아서 보내주는게 더 나을까요??

앱은 어떻게 처리해야 될 지 몰라서 우선 body에 담았는데 한번 찾아볼게요!!

choiseoji · 2025-06-29T14:08:39Z

src/main/java/run/backend/global/util/JwtTokenProvider.java

+    public String generateSignupToken(String providerId, String provider, String email, String name) {
+        long now = (new Date()).getTime();
+        return Jwts.builder()
+            .setSubject(providerId)
+            .claim("provider", provider)
+            .claim("email", email)
+            .claim("name", name)
+            .setExpiration(new Date(now + 600000))
+            .signWith(key, SignatureAlgorithm.HS256)
+            .compact();


이건 그냥 저희 signup api 를 호출하기 위한 토큰인거죠 ??

맞아요 추가 정보 받을 때 사용할 임시 토큰이용

choiseoji · 2025-06-29T14:20:39Z

src/main/java/run/backend/domain/auth/controller/AuthController.java

+        TokenResponse response = authService.completeSignup(signupRequest);
+
+        return ResponseEntity.ok(new CommonResponse<>("회원가입이 완료되었습니다.", response));
+    }


accessToken 만료 되었을 때 재발급하는 api 가 없어서 이것만 있으면 될거 같아요!!
accessToken 발급할때 refeshToken도 새로 발급하는 RTR 방식 사용해도 좋을거 같아요!!

재발급 api 까먹,, RTR 방식 찾아보고 수정할게요!!

choiseoji · 2025-06-29T14:32:54Z

src/main/java/run/backend/domain/auth/dto/response/SignupResponse.java

+public record SignupResponse(boolean isNewUser, String signupToken, String email, String name,
+                     String provider, TokenResponse tokens) {
+
+    public static SignupResponse forExistingUser(TokenResponse tokens) {
+        //기존 회원은 토큰만 리턴
+        return new SignupResponse(false, null, null, null, null, tokens);
+    }
+
+    public static SignupResponse forNewUser(String signupToken, String email, String name,
+        String provider) {
+        //신규 회원은 회원가입을 위한 정보와 임시 토큰 리턴
+        return new SignupResponse(true, signupToken, email, name, provider, null);
+    }
+}


null이 많긴 하지만 좋은거 같아요!!
null 없이 하려면 인터페이스 만들어서 forExistingUser 용 response, forNewUser 용 response 를 따로 구현하는 방법도 추천받았는데, 지금 방식이 더 나은 것 같아요

choiseoji · 2025-06-29T14:33:47Z

src/main/java/run/backend/domain/member/entity/Member.java

+    private LocalDateTime createdAt;
+
+    private LocalDateTime updatedAt;
+
+    private LocalDateTime deletedAt;


아 이거 baseEntity 만들어서 extends 하면 되는거라서
제가 엔티티 만들면서 추가해두겠습니다!!

choiseoji · 2025-06-29T14:35:09Z

src/main/java/run/backend/domain/auth/dto/request/SignupRequest.java

+
+import run.backend.domain.member.enums.Gender;
+
+public record SignupRequest(String signupToken, String nickname, Gender gender, int age, String profileImage) {


SignupResponse에 이미지 관련 필드는 없는거 같은데 profileImage는 어디서 받아오는 건가요??
만약, 유저가 직접 업로드하는 거라면 파일로 받아서 처리해야 될거 같아요!

이거 구글에서 제공해주는 사용자 프로필 생각했는데 그거 말고 저희 추가 정보 받는 곳에서 파일로 받도록 수정할게요!

github-actions · 2025-06-30T14:58:13Z

Test Coverage Report

Overall Project	53.21% `-39.65%`	🍏
Files changed	57.3%	🍏

File	Coverage
SignupRequest.java	100%	🍏
SignupResponse.java	100%	🍏
TokenResponse.java	100%	🍏
Role.java	100%	🍏
OAuthType.java	100%	🍏
Gender.java	100%	🍏
Member.java	100%	🍏
AuthService.java	81.51% `-18.49%`	🍏
RefreshToken.java	70.59% `-29.41%`	🍏
FileService.java	9.84% `-90.16%`	❌
FileController.java	0%	❌
CodeRequest.java	0%	❌
AuthController.java	0%	❌

choiseoji · 2025-06-30T15:06:49Z

src/main/java/run/backend/domain/auth/controller/AuthController.java

+
+    @PostMapping("/refresh")
+    public ResponseEntity<CommonResponse<TokenResponse>> refresh(
+        @RequestHeader("Authorization") String authorizationHeader) {


오 대박 이렇게 가지고 올 수 있는거 처음 알았어요

github-actions · 2025-06-30T15:27:44Z

Test Coverage Report

Overall Project	53.21% `-39.65%`	🍏
Files changed	57.3%	🍏

File	Coverage
SignupRequest.java	100%	🍏
SignupResponse.java	100%	🍏
TokenResponse.java	100%	🍏
Role.java	100%	🍏
OAuthType.java	100%	🍏
Gender.java	100%	🍏
Member.java	100%	🍏
AuthService.java	81.51% `-18.49%`	🍏
RefreshToken.java	70.59% `-29.41%`	🍏
FileService.java	9.84% `-90.16%`	❌
FileController.java	0%	❌
CodeRequest.java	0%	❌
AuthController.java	0%	❌

west-eastH added 10 commits June 26, 2025 16:42

[#7] feat: oauth2-client 의존성 추가

bde5bb8

[#4] feat: 데이터베이스 연결

0eed631

[#7] feat: 멤버 엔티티 생성

8ed4711

[#7] feat: oauth2-client 설정

413afa2

[#7] feat: 시큐리티 설정

8471148

[#7] feat: JWT 관련 의존성 추가, 유틸 클래스 추가

ca98821

[#7] feat: 멤버 엔티티 정의

82a66d3

[#7] feat: OAuth2 유저 정보 인터페이스 및 클래스 정의

c31b69c

[#7] feat: OAuth2 회원가입 및 로그인 구현

9559084

[#7] test: OAuth2 로그인 관련 테스트 코드 작성

4491559

west-eastH requested a review from choiseoji June 29, 2025 13:15

west-eastH self-assigned this Jun 29, 2025

west-eastH added the ✨feat 기능 개발 시 label Jun 29, 2025

west-eastH linked an issue Jun 29, 2025 that may be closed by this pull request

✨feat : OAuth2 회원가입 #7

Closed

2 tasks

choiseoji approved these changes Jun 29, 2025

View reviewed changes

west-eastH added 3 commits June 30, 2025 14:53

[#7] feat: 토큰 재발급(RTR) 구현

70fd898

[#7] test: 토큰 재발급(RTR) 구현

f5ae2bb

[#7] feat: 프로필 이미지 업로드 & 조회

78a4e2f

choiseoji approved these changes Jun 30, 2025

View reviewed changes

[#7] fix: 엔드포인트 /api/v1 통일

67d1930

west-eastH merged commit 6631c64 into develop Jun 30, 2025
1 check passed

west-eastH deleted the feat/7-oauth2 branch June 30, 2025 15:28

		public record TokenResponse(String accessToken, String refreshToken) {

		}


		import run.backend.domain.member.enums.Gender;

		public record SignupRequest(String signupToken, String nickname, Gender gender, int age, String profileImage) {

Conversation

west-eastH commented Jun 29, 2025

✨ 연관된 이슈

📝 작업 내용 (주요 변경 사항)

💬 리뷰 요구사항

Uh oh!

github-actions bot commented Jun 29, 2025

Test Coverage Report

Uh oh!

choiseoji left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Jun 30, 2025

Test Coverage Report

Uh oh!

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Jun 30, 2025

Test Coverage Report

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants