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

[이준현] 프리코스 미션 제출합니다. #15

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

dlwnsgus07
Copy link

No description provided.

이준현 added 8 commits November 3, 2024 14:53
username, password를 바탕으로 사용자를 식별하는 FormLoginInterceptor 구현
Basic Token을 디코딩하여 사용자 정보를 가져오는 BasicTokenDecoder 구현
Authorization Header의 Basic 토큰을 가져와 사용자를 식별하는 BasicAuthInterceptor 구현
Basic Token Prefix, Context_key 등 여러 클래스에서 사용하는 상수들을 하나의 클래스에서 관리할 수 있도록 변경
userDetailService의 구현체에서 관리하던 사용자 인증 로직을 security 하위 패키지에 있는 클래스들이 사용하도록 변경

refactor : 잘못된 줄바꿈 수정

불필요한 줄바꿈 제거
FormLoginAuthInterceptor의 parameter 정보와 세션정보를 검증하는 로직과 사용자가 맞는지 인증하는 로직을 별도의 메서드 분리

feat : SecurityConstants 빈 생성자 추가

해당 클래스가 오로지 상수용 클래스로만 사용될 수 있도록 private 빈 생성자 추가
BasicTokenDecoder가 app이 아닌 sercurity 패키지에 속하도록 변경하고 컴포넌트 스캔 범위를 app, security 패키지를 사용할 수 있게 SecurityAuthenticationApplication 위치 변경
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.

1 participant