Skip to content

#07 로그인 #37

Merged
kingkang85 merged 2 commits intodevelopfrom
20-7-로그인
Dec 15, 2024

Hidden character warning

The head ref may contain hidden characters: "20-7-\ub85c\uadf8\uc778"
Merged

#07 로그인 #37
kingkang85 merged 2 commits intodevelopfrom
20-7-로그인

Conversation

@kingkang85
Copy link
Copy Markdown
Contributor

📋 요약

  • 로그인 기능과 JWT 토큰 발급 및 인증 구현

🛠 변경 사항

  • 이번 Pull Request에서 작업한 주요 변경 사항은 다음과 같습니다:
    • 새로운 기능 추가 또는 버그 수정
    • 코드 리팩토링 또는 최적화
    • 문서 업데이트

🔗 관련 이슈

📸 스크린샷 또는 GIF (해당되는 경우)

  • 로그인
    image
    로그인 요청을 보내면, Headers에 있는 Authorization에 JWT 토큰이 담겨 있습니다.

  • 인증
    image
    특정 경로에 요청을 보낼 때 발급받은 토큰을 Headers에 담아 보내면, 토큰 검증 후 해당 사용자를 인증된 상태로 만듭니다.

✅ 체크리스트

  • 코드가 정상적으로 동작하는지 테스트했습니다.
  • 관련 문서를 작성하거나 업데이트했습니다. (해당되는 경우)
  • 변경 사항을 반영한 테스트 코드를 추가했습니다.
  • 모든 테스트가 성공적으로 통과했습니다.

🛡 테스트 방법

.

📚 추가 참고 사항

.

- LoginFilter: 클라이언트가 보낸 아이디와 비밀번호 받음
- CustomUserDetailsService: 사용자 정보 조회
- CustomUserDetails: 사용자 정보 포맷

Related to: #20
- JWTUtil: 토큰 생성 및 검증 기능 구현
- LoginFilter: 로그인 성공 시 JWT 토큰 발급
- JWTFilter: 요청 시 JWT 토큰 검증 및 인증 처리

Resolves: #20
Copy link
Copy Markdown
Contributor

@LeeGukgeon LeeGukgeon left a comment

Choose a reason for hiding this comment

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

security 의 JWT 방식으로 로그인 기능 구현 확인했습니다 pull 받아서 어떤 구조로 되어있는지 확인해보고 싶네요

Copy link
Copy Markdown
Contributor

@KIM9909 KIM9909 left a comment

Choose a reason for hiding this comment

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

와우 대박이네요 굿굿 !!!

@uni-zun
Copy link
Copy Markdown
Contributor

uni-zun commented Dec 15, 2024

어려운 부분 잘 하고 계시네요 굿굿 !!

@yunjaeeun
Copy link
Copy Markdown
Contributor

로그인시 userPk를 전달해주면 더욱 좋을 것 같네요

Copy link
Copy Markdown
Contributor

@MinFe4990 MinFe4990 left a comment

Choose a reason for hiding this comment

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

!check

@kingkang85 kingkang85 merged commit 21ea342 into develop Dec 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants