Skip to content

Commit 9ee1286

Browse files
authored
Merge pull request #347 from mosu-dev/develop
[배포] kakao INVALID_TOKEN 에러 처리
2 parents 3faffd9 + dce1869 commit 9ee1286

4 files changed

Lines changed: 18 additions & 19 deletions

File tree

src/main/java/life/mosu/mosuserver/global/config/SecurityConfig.java

Lines changed: 9 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,15 @@
44
import java.util.HashMap;
55
import java.util.List;
66
import java.util.Map;
7-
7+
import life.mosu.mosuserver.application.oauth.OAuthUserService;
8+
import life.mosu.mosuserver.global.filter.TokenExceptionFilter;
9+
import life.mosu.mosuserver.global.filter.TokenFilter;
10+
import life.mosu.mosuserver.global.handler.AuthLogoutHandler;
11+
import life.mosu.mosuserver.global.handler.AuthLogoutSuccessHandler;
12+
import life.mosu.mosuserver.global.handler.OAuth2LoginFailureHandler;
13+
import life.mosu.mosuserver.global.handler.OAuth2LoginSuccessHandler;
14+
import life.mosu.mosuserver.global.resolver.AuthorizationRequestRedirectResolver;
15+
import lombok.RequiredArgsConstructor;
816
import org.springframework.context.annotation.Bean;
917
import org.springframework.context.annotation.Configuration;
1018
import org.springframework.http.HttpMethod;
@@ -31,16 +39,6 @@
3139
import org.springframework.web.cors.CorsConfigurationSource;
3240
import org.springframework.web.cors.UrlBasedCorsConfigurationSource;
3341

34-
import life.mosu.mosuserver.application.oauth.OAuthUserService;
35-
import life.mosu.mosuserver.global.filter.TokenExceptionFilter;
36-
import life.mosu.mosuserver.global.filter.TokenFilter;
37-
import life.mosu.mosuserver.global.handler.AuthLogoutHandler;
38-
import life.mosu.mosuserver.global.handler.AuthLogoutSuccessHandler;
39-
import life.mosu.mosuserver.global.handler.OAuth2LoginFailureHandler;
40-
import life.mosu.mosuserver.global.handler.OAuth2LoginSuccessHandler;
41-
import life.mosu.mosuserver.global.resolver.AuthorizationRequestRedirectResolver;
42-
import lombok.RequiredArgsConstructor;
43-
4442
@Configuration
4543
@EnableWebSecurity
4644
@EnableMethodSecurity

src/main/java/life/mosu/mosuserver/global/config/WebMvcConfig.java

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,16 +1,14 @@
11
package life.mosu.mosuserver.global.config;
22

33
import java.util.List;
4-
4+
import life.mosu.mosuserver.global.resolver.PhoneNumberArgumentResolver;
5+
import life.mosu.mosuserver.global.resolver.UserIdArgumentResolver;
6+
import lombok.RequiredArgsConstructor;
57
import org.springframework.context.annotation.Configuration;
68
import org.springframework.web.method.support.HandlerMethodArgumentResolver;
79
import org.springframework.web.servlet.config.annotation.CorsRegistry;
810
import org.springframework.web.servlet.config.annotation.WebMvcConfigurer;
911

10-
import life.mosu.mosuserver.global.resolver.PhoneNumberArgumentResolver;
11-
import life.mosu.mosuserver.global.resolver.UserIdArgumentResolver;
12-
import lombok.RequiredArgsConstructor;
13-
1412
@Configuration
1513
@RequiredArgsConstructor
1614
public class WebMvcConfig implements WebMvcConfigurer {

src/main/java/life/mosu/mosuserver/global/handler/OAuthErrorType.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,8 @@
66
public enum OAuthErrorType {
77
CANCELED("CANCELED"),
88
DUPLICATE("DUPLICATE"),
9-
UNKNOWN("UNKNOWN");
9+
UNKNOWN("UNKNOWN"),
10+
INVALID_TOKEN("INVALID_TOKEN");
1011

1112
private final String message;
1213

@@ -21,6 +22,8 @@ public static OAuthErrorType from(String text) {
2122
return switch (text) {
2223
case "DUPLICATE" -> DUPLICATE;
2324
case "[access_denied] User denied access" -> CANCELED;
25+
case "[invalid_token_response] An error occurred while attempting to retrieve the OAuth 2.0 Access Token Response: 401 : [no body]" ->
26+
INVALID_TOKEN;
2427
default -> UNKNOWN;
2528
};
2629
}

src/main/java/life/mosu/mosuserver/presentation/profile/ProfileController.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public ResponseEntity<ApiResponseWrapper<Void>> create(
3737
}
3838

3939
@PutMapping
40-
@PreAuthorize("isAuthenticated() and hasRole('USER')")
40+
@PreAuthorize("hasRole('USER')")
4141
public ResponseEntity<ApiResponseWrapper<Void>> update(
4242
@UserId Long userId,
4343
@Valid @RequestBody EditProfileRequest request
@@ -47,7 +47,7 @@ public ResponseEntity<ApiResponseWrapper<Void>> update(
4747
}
4848

4949
@GetMapping
50-
@PreAuthorize("isAuthenticated() and hasRole('USER')")
50+
@PreAuthorize("hasRole('USER')")
5151
public ResponseEntity<ApiResponseWrapper<ProfileDetailResponse>> getProfile(
5252
@UserId Long userId
5353
) {

0 commit comments

Comments
 (0)