Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

9주차 미션 / 서버 2조 강상민 #7

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

isoo127
Copy link

@isoo127 isoo127 commented May 31, 2024

인증 인가 및 페이징 처리 구현 완료

의문점
argument resolver에서 has annotation 메소드가 항상 false를 반환하였는데 원인을 찾지 못하였음...
-> 항상 true 값으로 임의로 일단은 해결

import lombok.extern.slf4j.Slf4j;
import org.springframework.core.MethodParameter;
import org.springframework.http.HttpHeaders;
import org.springframework.security.access.prepost.PreAuthorize;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

resolver에서 현재 hasAnnotation으로 해당 어노테이션인지를 확인을 합니다. 하지만 현재 import 해오는 클래스를 보면 저희가 만든 어노테이션이 아닌 spring security에서 제공하는 어노테이션을 가져와서 사용하고 있습니다. 아마 두 개의 어노테이션이 다르기에 false가 나오는 문제가 발생하는 것으로 보입니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants