two-clock / backend

개발자를 위한 Github 계정 및 저장소 정보 공유 서비스
4 stars 0 forks source link

feat: Implement GitHub Login with Access Token #24

Closed openmpy closed 3 months ago

openmpy commented 3 months ago

관련 이슈

변경 사항

스크린샷 2024-08-06 오후 8 57 00

체크 목록

리뷰 요청

@jjangsky 님~ 로그인 작업 완료 했습니다. 추후에 Refresh Token 과 로그아웃 작업까지 마무리 할 예정입니다. Swagger 주소는 http://localhost:8082/swagger-ui.html 입니다. 확인 부탁드리겠습니다. 감사합니다.

openmpy commented 3 months ago

@openmpy 님 개발하시느라 고생이 많으십니다. 스프링 시큐리티 관련 지식이 부족해서 리뷰하면서 공부하고 많은 도움이 되는 시간이 된 것 같습니다.

추가적으로 인증 실패에 대한 로직은 어디 부분에 있는지 알 수 있을까요?

@jjangsky 님~ 인증 실패에 대한 로직은 JwtExceptionFilter 이 부분에 있습니다. 요청이 정상적이라면 필터를 타게 되고, 정상적이지 않다면 UNAUTHORIZED 를 반환하게 됩니다.

openmpy commented 3 months ago

@jjangsky 님~ PR Merge 진행 하도록 하겠습니다. 리뷰 감사합니다.