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

Feature_1/security, auth 구현 #3

Merged
merged 9 commits into from
Mar 29, 2024
Merged

Conversation

Ryeolee
Copy link
Contributor

@Ryeolee Ryeolee commented Mar 23, 2024

  1. gradle 추가
  • implementation 'org.springframework.boot:spring-boot-starter-data-redis'
  • testImplementation 'it.ozimov:embedded-redis:0.7.2'
  • implementation 'org.springframework.boot:spring-boot-starter-webflux'
  1. security 적용
  • 토큰 검증 유무를 configure()와 filterChain()로 나눔
  1. api 개발
  • 구글, 카카오 로그인
  • 로그아웃
  • 토큰 재발급

@Ryeolee Ryeolee linked an issue Mar 23, 2024 that may be closed by this pull request
@Ryeolee Ryeolee changed the base branch from main to dev March 23, 2024 13:05
@MinchoGreenT
Copy link
Contributor

  1. 레디스 관련 정보들을 application.yml에 추가하신 듯 한데, 수정사항이 있다면 노션의 해당 페이지의 코드도 수정 후 언급 부탁드립니다!
  2. SocialLogin 클래스의 verifySocialUserData 메서드의 경우 반환값이 사용되지 않는 것으로 보입니다. 유저가 존재하지 않는 경우의 에러를 잡으려고 하신 것 같은데, 맞을까요?

@Ryeolee
Copy link
Contributor Author

Ryeolee commented Mar 25, 2024

네~ 소셜에서 유저에 대한 정보를 받아오고 데이터 체크를 위한 함수입니답

      2. socialLogin 클래스 validateUserDataAndSignUp -> socialSignUp 함수 변경 orElseThrow 적용
@MinchoGreenT
Copy link
Contributor

reissueToken 메서드에서 validateRefreshToken 메서드의 반환값이 사용되고 있지 않습니다
-> 토큰에 오류가 있다면 응답을 반환하지만 아무일도 일어나지 않습니다

이부분은 예외 클래스를 하나 만들어서 AuthExceptionHandler로 보내야 할 것 같네요
함수의 반환값을 void로 하고 if문에서 오류가 날 경우 예외를 던지는 식으로요!

그부분만 수정하고 바로 머지해도 될 것 같습니다.

@Ryeolee Ryeolee merged commit af393f3 into dev Mar 29, 2024
1 check passed
@Ryeolee Ryeolee deleted the feature_1/security,-auth-구현 branch March 29, 2024 05:07
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.

[FEAT} 시큐리티 적용 및 auth 관련 api
2 participants