Skip to content

Commit a239ed8

Browse files
authored
Merge pull request #120 from TeamLearningFlow/develop
Develop
2 parents 3e1639a + f6cda14 commit a239ed8

File tree

5 files changed

+14
-11
lines changed

5 files changed

+14
-11
lines changed

src/main/java/learningFlow/learningFlow_BE/security/handler/OAuth2LoginSuccessHandler.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,9 @@ public class OAuth2LoginSuccessHandler implements AuthenticationSuccessHandler {
2727

2828
private final JwtTokenProvider jwtTokenProvider;
2929

30-
@Value("${app.frontend-url}") // added
31-
private String frontendUrl; // added
30+
@Value("${app.frontend-url}")
31+
private String frontendUrl;
32+
3233
@Override
3334
public void onAuthenticationSuccess(HttpServletRequest request, HttpServletResponse response,
3435
Authentication authentication) throws IOException {

src/main/java/learningFlow/learningFlow_BE/service/auth/oauth/OAuth2UserRegistrationService.java

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ public Map<String, Object> getAdditionalInfoRequirements() {
4444
Map<String, Object> response = new HashMap<>();
4545
response.put("message", "추가 정보 입력이 필요합니다");
4646
response.put("requiredFields", Arrays.asList(
47-
"job", "interestFields", "gender", "preferType"
47+
"name", "job", "interestFields", "preferType"
4848
));
4949

5050
return response;
@@ -64,15 +64,14 @@ public UserResponseDTO.UserLoginResponseDTO updateAdditionalInfo(
6464

6565
Claims claims = jwtTokenProvider.getClaims(temporaryToken);
6666
String email = claims.getSubject();
67-
String name = claims.get("name", String.class);
6867
String providerId = claims.get("providerId", String.class);
6968
SocialType socialType = SocialType.valueOf(claims.get("socialType", String.class));
7069

7170

7271
User newUser = User.builder()
7372
.loginId(socialType.name() + "_" + providerId)
7473
.email(email)
75-
.name(name)
74+
.name(additionalInfo.getName())
7675
.providerId(providerId)
7776
.pw("OAUTH2_USER")
7877
.socialType(socialType)

src/main/java/learningFlow/learningFlow_BE/web/controller/LoginController.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -150,15 +150,16 @@ public ApiResponse<?> getAdditionalInfoPage() {
150150
OAuth2 회원가입의 추가 정보를 입력받습니다.
151151
152152
[필수 입력]
153-
- 직업
154-
- 관심분야 (다중선택)
155-
- 선호 미디어 타입
153+
- 이름: 실명 또는 닉네임
154+
- 직업: STUDENT, ADULT, EMPLOYEE, JOB_SEEKER, OTHER
155+
- 관심분야: 다중선택 (APP_DEVELOPMENT, WEB_DEVELOPMENT, PROGRAMMING_LANGUAGE, DEEP_LEARNING, STATISTICS, DATA_ANALYSIS, UI_UX, PLANNING, BUSINESS_PRODUCTIVITY, FOREIGN_LANGUAGE, CAREER)
156+
- 선호 미디어: VIDEO, TEXT
156157
157158
[선택 입력]
158159
- 프로필 이미지 URL
159160
160161
[주의사항]
161-
- 이메일/이름은 구글 계정 정보 사용
162+
- 이메일은 구글 계정 정보 사용
162163
- 이미지 미입력시 기본 이미지 사용
163164
""")
164165
public ApiResponse<UserResponseDTO.UserLoginResponseDTO> updateAdditionalInfo(

src/main/java/learningFlow/learningFlow_BE/web/dto/user/UserRequestDTO.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,10 @@ public static class UserLoginDTO {
5959

6060
@Getter
6161
public static class AdditionalInfoDTO {
62+
63+
@NotBlank(message = "이름은 필수 입력값입니다")
64+
String name;
65+
6266
@NotNull(message = "직업은 필수 선택값입니다")
6367
Job job;
6468

src/main/resources/application.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,6 @@ spring:
2424
client:
2525
registration:
2626
google:
27-
redirect-uri: "http://onboarding.p-e.kr:8080/login/oauth2/code/google"
2827
client-id: ${SPRING_SECURITY_OAUTH2_CLIENT_REGISTRATION_GOOGLE_CLIENT_ID}
2928
client-secret: ${SPRING_SECURITY_OAUTH2_CLIENT_REGISTRATION_GOOGLE_CLIENT_SECRET}
3029
scope:
@@ -59,7 +58,6 @@ custom:
5958
app:
6059
frontend-url: http://localhost:3000 #http://54.180.118.227:8080
6160

62-
6361
server:
6462
port: 8080
6563
compression:

0 commit comments

Comments
 (0)