Feature/ 소셜 로그인 구현, Filter 내용 추가 by KII1ua · Pull Request #35 · JECT-Study/JECT2-4th-Server

KII1ua · 2026-03-30T16:58:32Z

📌 관련 이슈

closes 소셜로그인 기능 추가 #33

🔍 작업 내용

OAuth2를 사용하여 소셜 로그인 기능 구현, 로그인 성공 후 accessToken, refreshToken 발급 아직 배치를 사용하진 않았음

📝 변경 사항

OAuth2UserService : 구글로부터 전달받은 사용자 정보를 서비스에 맞게 작성
OAuthAttributes : 구글로부터 응답데이터를 DTO로 변환
SuccessHandler : 로그인 성공시 AuthService를 호출하여 토큰 생성 -> 프론트엔드 지정 주소로 토큰을 담아 새로고침

💬 리뷰어에게

release: develop → main

tlarbals824 · 2026-04-05T07:13:55Z

src/main/java/com/ject/vs/config/OAuth2LoginSuccessHandler.java

+                .queryParam("accessToken", tokenResponse.getAccessToken())
+                .queryParam("refreshToken", tokenResponse.getRefreshToken())
+                .build()


URL 쿼리 스트링은 브라우저 히스토리, Nginx access log, 프록시 서버 로그에 기록됩니다. 일회용 authorization code를 발급하고 프론트에서 POST로 교환하는 방식이나, secure HttpOnly 쿠키 방식은 어떻게 생각하시나요?

넵! httponly 방식을 사용해서 수정하겠습니다

tlarbals824 · 2026-04-05T07:15:28Z

src/main/java/com/ject/vs/config/OAuth2LoginSuccessHandler.java

+        OAuth2User oauth2User = (OAuth2User) authentication.getPrincipal();
+
+        String registrationId = oauthToken.getAuthorizedClientRegistrationId();
+        String userNameAttributeName = "sub";


혹시 해당 변수는 어디에서 사용되는걸까요?

구글 소셜로그인에서는 유저마다 구분하는 키값이 sub 값으로 구분해서 해당 변수를 사용해서 키값으로 지정할 생각이였습니다.

tlarbals824 · 2026-04-05T07:19:20Z

src/main/java/com/ject/vs/domain/Token.java

+public class Token {
+    @Id @GeneratedValue(strategy = GenerationType.IDENTITY)
+    private Long id;
+
+    private String tokenId;


TokenId를 통해 토큰이 존재하는지 검증할 것 같은데, 혹시 인덱스를 추가하시지 않은 이유가 있을까요?

앗 hibernate가 자동으로 기본키값을 인덱스로 생성해주는줄 알았는데 아니네요!! 이후에 수정하겠습니다 좋은 정보 감사드려요!

tlarbals824 · 2026-04-05T07:20:38Z

src/main/java/com/ject/vs/config/OAuth2LoginSuccessHandler.java

+
+@Component
+@RequiredArgsConstructor
+public class OAuth2LoginSuccessHandler extends SimpleUrlAuthenticationSuccessHandler {


소셜 로그인이 동작하는지 테스트해보셨을까요?

넵 테스트 했을때 토큰값도 정상적으로 쿼리에 잘 들어가는것까지 확인했습니다.

tlarbals824 · 2026-04-05T07:21:53Z

src/main/java/com/ject/vs/service/AuthService.java

+        LocalDateTime accessExpiresAt = jwtProvider.getClaims(accessToken)
+                .getExpiration()
+                .toInstant()
+                .atZone(ZoneId.systemDefault())
+                .toLocalDateTime();
+
+        LocalDateTime refreshExpiresAt = jwtProvider.getClaims(refreshToken)
+                .getExpiration()
+                .toInstant()
+                .atZone(ZoneId.systemDefault())
+                .toLocalDateTime();


토큰을 만들고, 다시 파싱해서 만료 시간을 추출하시는 이유가 있을까요? 토큰이 생성될 때, TokenInfo와 같은 값 객체를 활용하면 필요한 정보를 한번에 반환할 수 있어 보여요

오 넵!! 해당 방법을 사용해서 리팩토링 해보겠습니다

tlarbals824 · 2026-04-05T07:22:24Z

src/main/java/com/ject/vs/util/JwtProvider.java

+    private SecretKey secretKey;
+
+    @PostConstruct
+    public void Init() {        // secretkey 값을 읽어 디코딩 하여 바이트 배열로 변환 후 hmac으로 암호화


Java 컨밴션에서는 함수가 소문자로 시작하는데, 혹시 대문자로하신 이유가 있을까요?

tlarbals824 · 2026-04-05T07:22:40Z

src/main/java/com/ject/vs/util/JwtProvider.java

+        this.secretKey = Keys.hmacShaKeyFor(keyBytes);
+    }
+
+    public String createAccesssToken(Long userId, String tokenId) {


오타가 있어보여요! createAccesssToken -> createAccessToken

넵 수정하겠습니다!

tlarbals824 · 2026-04-05T07:23:28Z

src/main/java/com/ject/vs/service/AuthService.java

+        Token accesTk = Token.builder()
+                .tokenId(accessTokenId)
+                .user(user)
+                .tokenValue(accessToken)
+                .tokenType(TokenType.ACCESS)
+                .expiresAt(accessExpiresAt)
+                .revoked(false)
+                .build();
+
+        Token refreshTk = Token.builder()
+                .tokenId(refreshTokenId)
+                .user(user)
+                .tokenValue(refreshToken)
+                .tokenType(TokenType.REFRESH)
+                .expiresAt(refreshExpiresAt)
+                .revoked(false)
+                .build();


약어는 사용하지 않는게 좋을 것 같아요! 물론 초반에 저희가 리뷰를 다 진행하다보니 TK = Token이라고 인지가 되지만, 이후에는 살짝 인지 부하가 발생할 것 같아서요!

넵 알겠습니다 수정하겠습니다.

tlarbals824 · 2026-04-05T07:24:17Z

src/main/java/com/ject/vs/domain/User.java

+import lombok.Setter;
+
+@Entity
+@Getter


Setter는 선언하지 않는게 좋아보여요! 만일 값이 수정되어야한다면 목적이 들어나는 함수를 정의하도록 하는게 좋아보여요

넵 수정하겠습니다.

tlarbals824 · 2026-04-05T07:25:14Z

src/main/java/com/ject/vs/util/JwtProvider.java

+                .compact();
+    }
+
+    public String createRefreshToken(Long userId, String tokenId) {


createAccesssToken와 createRefreshToken가 거의 동알한데 통합할 수 없을까요?

tlarbals824 · 2026-04-05T07:25:41Z

src/main/java/com/ject/vs/service/AuthService.java

+                .revoked(false)
+                .build();
+
+        tokenRepository.save(accesTk);


혹시 AccessToken을 저장하는 이유가 있을까요?

tlarbals824 · 2026-04-05T07:26:54Z

src/main/java/com/ject/vs/util/JwtProvider.java

+            Jwts.parser()
+                    .verifyWith(secretKey)
+                    .build()


매번 인스턴스를 생성하기보다는 PostContruct에서 한 번 만들어서 재활용할 수 있을 것 같아요

tlarbals824 · 2026-04-05T07:27:29Z

src/main/java/com/ject/vs/service/CustomOAuth2UserService.java

+public class CustomOAuth2UserService implements OAuth2UserService<OAuth2UserRequest, OAuth2User> {
+    @Override
+    public OAuth2User loadUser(OAuth2UserRequest userRequest) throws OAuth2AuthenticationException {
+        OAuth2UserService<OAuth2UserRequest, OAuth2User> delegate = new DefaultOAuth2UserService();


매번 인스턴스화하는 이유가 있을까요?

tlarbals824 · 2026-04-05T07:30:27Z

src/main/java/com/ject/vs/domain/Token.java

+    @Lob
+    private String tokenValue;


@lob 대신 @column(length = 1024) 같은 VARCHAR 타입은 어떨까요? JWT 문자열이 보통 수백 바이트 수준이라 VARCHAR로 충분하고, CLOB/TEXT는 일부 DB에서 row 외부에 별도 저장되어 조회 시 추가 I/O가 발생할 수 있어서요!

tlarbals824 · 2026-04-05T07:31:00Z

src/main/java/com/ject/vs/dto/LoginTokenResponse.java

+
+@Getter
+@Builder
+public class LoginTokenResponse {


Record를 활용할 수 있어보여요!

…수정사항등 수정완료

tlarbals824 and others added 13 commits March 15, 2026 23:35

Merge pull request #12 from JECT-Study/develop

ef47a21

release: develop → main

Merge pull request #18 from JECT-Study/develop

7200690

release: develop → main

Merge pull request #20 from JECT-Study/develop

27af347

release: develop → main

Merge pull request #23 from JECT-Study/develop

929b3f6

release: develop → main

Merge pull request #25 from JECT-Study/develop

a60b4d4

release: develop → main

Merge pull request #27 from JECT-Study/develop

f4d4c6e

release: develop → main

Merge pull request #29 from JECT-Study/develop

f11496e

release: develop → main

jwt 의존성 추가, google api 키 추가

871345f

Token클래스 , TokenType 열거형 추가, 토큰 생성 util 추가

d8be6ed

Google 소셜로그인 DTO 생성

0fdc04f

Google 소셜로그인 OAuth 서비스, 소셜로그인 핸들러 추가

cbfa8e6

SecurityConfig 수정, yml파일 수정

0178a54

google-yml파일 추적 삭제

38c94cd

KII1ua self-assigned this Mar 30, 2026

KII1ua requested review from Junhyukkkk and tlarbals824 as code owners March 30, 2026 16:58

KII1ua added the feature label Mar 30, 2026

tlarbals824 reviewed Apr 5, 2026

View reviewed changes

KII1ua added 3 commits April 12, 2026 22:23

로직 수정

862f380

기존 queryParameter에서 httponly cookie로직 변경

74ecdb6

cookie filter 추가, AuthController 추가 -> accesstoken이 만료됬을 경우 재발급, 그 외 …

ba0dc59

…수정사항등 수정완료

Conversation

KII1ua commented Mar 30, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📌 관련 이슈

🔍 작업 내용

📝 변경 사항

💬 리뷰어에게

Uh oh!

Choose a reason for hiding this comment

Uh oh!

KII1ua Apr 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

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!

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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

KII1ua commented Mar 30, 2026 •

edited

Loading

KII1ua Apr 6, 2026 •

edited

Loading