로그인 Interceptor 개선

[BGuga]

#️⃣연관된 이슈

#1612

📝작업 내용

@GetMapping("/sessions/{sessionId}/keywords/{keywordId}/quizs/{quizId}")
    public ResponseEntity<QuizResponse> findQuizById(@PathVariable Long quizId,
        @AuthMemberPrincipal LoginMember member) {
        return ResponseEntity.ok(quizService.findById(quizId, member.getId()));
    }

@DeleteMapping("/sessions/{sessionId}/keywords/{keywordId}/quizs/{quizId}")
    public ResponseEntity<Void> deleteQuiz(@PathVariable Long sessionId,
                                           @PathVariable Long keywordId,
                                           @PathVariable Long quizId) {
        quizService.deleteQuiz(quizId);
        return ResponseEntity.noContent().build();
    }

위와 같은 상황에서 Delete method에서도 로그인 검증이 일어나는 문제가 있었습니다.
이를 MethodPattern 이란 객체를 생성하여 메서드와 uri 두 가지가 만족해야 검증하도록 변경했습니다.

[sonarqubecloud]

SonarCloud Quality Gate failed.   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
2 Code Smells

67.6% Coverage
0.0% Duplication

The version of Java (11.0.21) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

[amaran-th]

안녕하세요 푸우!
아까 설명해주신 덕분에 코드 이해가 한 결 쉬웠습니다ㅎㅎ
어려운 작업이었을텐데 수고 많으셨어요.
전체적으로 크게 수정할만한 부분은 보이지 않는 것 같습니다. 테스트코드도 꼼꼼하게 작성해주셨구요.
그러니 approve하도록 하겠습니다!

[amaran-th]

오 상태코드도 적절하게 수정해주셨네요! 굿입니다^^*