Skip to content

Commit 637a162

Browse files
authored
Merge pull request #102 from Cod2-Crafters/refactor-interceptor
중복된 로그인 체크 로직 제거
2 parents 0abedf2 + ba7ca0a commit 637a162

File tree

2 files changed

+13
-26
lines changed

2 files changed

+13
-26
lines changed

src/main/java/com/codecrafter/commenting/config/WebServiceConfig.java

+13-15
Original file line numberDiff line numberDiff line change
@@ -23,25 +23,23 @@ public class WebServiceConfig implements WebMvcConfigurer {
2323
@Override
2424
public void addInterceptors(InterceptorRegistry registry) {
2525
registry.addInterceptor(loginCheckInterceptor)
26-
.addPathPatterns(
27-
"/api/member/setting/**", // 설정 수정
28-
"/api/recommends/**",
29-
"/api/conversations/question/update", // 질문수정
30-
"/api/conversations/question/*", // 질문 삭제
31-
"/api/conversations/answer", // 답변 작성
32-
"/api/conversations/answer/update", // 답변 수정
33-
"/api/conversations/answer/*", // 답변 삭제
34-
"/api/conversations/members/*/send", // 보낸질문 조회
35-
"/api/statistics" // 통계 조회
36-
)
37-
38-
;
26+
.addPathPatterns(
27+
"/api/member/setting/**", // 설정 수정
28+
"/api/recommends/**",
29+
"/api/conversations/question/update", // 질문수정
30+
"/api/conversations/question/*", // 질문 삭제
31+
"/api/conversations/answer", // 답변 작성
32+
"/api/conversations/answer/update", // 답변 수정
33+
"/api/conversations/answer/*", // 답변 삭제
34+
"/api/conversations/members/*/send", // 보낸질문 조회
35+
"/api/statistics" // 통계 조회
36+
);
3937

4038
registry.addInterceptor(recommendInterceptor)
41-
.addPathPatterns("/api/recommends/**");
39+
.addPathPatterns("/api/recommends/**");
4240

4341
registry.addInterceptor(conversationQuestionInterceptor)
44-
.addPathPatterns("/api/conversations/question");
42+
.addPathPatterns("/api/conversations/question");
4543
}
4644

4745
@Bean

src/main/java/com/codecrafter/commenting/controller/interceptor/RecommendInterceptor.java

-11
Original file line numberDiff line numberDiff line change
@@ -29,17 +29,6 @@ public class RecommendInterceptor implements HandlerInterceptor {
2929

3030
@Override
3131
public boolean preHandle(HttpServletRequest request, HttpServletResponse response, Object handler) throws Exception {
32-
Long currentMemberId = SecurityUtil.getCurrentMember().getId();
33-
// currentMemberId == 0 --> 로그인 안한것 --> 권한 없다.
34-
if (currentMemberId == 0L) {
35-
createResponseBody(
36-
response,
37-
new ApiResponse(ApiStatus.ERROR,"로그인이 필요한 서비스입니다.", null, null),
38-
HttpStatus.FORBIDDEN
39-
);
40-
return false;
41-
}
42-
4332
String requestMethod = request.getMethod();
4433

4534
if (requestMethod.equals("POST")) {

0 commit comments

Comments
 (0)