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

[BE] feat(#636) : 관리자 로그인 구현 #637

Merged
merged 4 commits into from
Oct 3, 2024

Conversation

rndudals
Copy link
Contributor

@rndudals rndudals commented Oct 1, 2024

#️⃣ 연관된 이슈

ex) #636

Pull Request Type

어떤 변경 사항이 있나요?

  • 새로운 기능 추가
  • 코드 리팩토링
  • 버그 수정
  • CSS 등 사용자 UI 디자인 변경
  • 테스트 추가, 테스트 리팩토링
  • 코드에 영향을 주지 않는 변경사항
    • 오타 수정, 문서 수정, 변수명 변경, 주석 추가 및 수정

Pull Request Checklist

PR이 다음 요구 사항을 충족하는지 확인하세요.

  • 변경 사항에 대한 테스트를 모두 통과했나요?
  • 코드 정렬은 적용했나요?
  • 새로운 branch에 작업 후 dev로 PR을 요청했나요?

📝 작업 내용

[배경]
엡 베포 테스트 시 관리자 권한으로 로그인이 필요

[구현]
관리자 Id, Password를 받아 무제한 jwt토큰을 넘겨준다.

관리자로 로그인

  • id, password 입력
POST: /auth/admin

완료

  • 로그인 성공 실패 로직 및 테스트 구현
    • id, password 검증
    • return JWT Token
  • yml 파일 변경(노션에 10/2 업데이트 해두었습니다.)
  • 서버 �시크릿 파일 업데이트

💬 리뷰 요구사항

놓친 점이나 고칠 점 피드백 부탁드립니다~!

Copy link

github-actions bot commented Oct 2, 2024

📝 테스트 커버리지 리포트

Overall Project 78.32% -0.02% 🍏
Files changed 95% 🍏

File Coverage
ExceptionMessage.java 100% 🍏
SecurityConfig.java 100% 🍏
AdminLoginRequest.java 87.76% -12.24% 🍏
AuthService.java 79.02% 🍏
AuthController.java 68.63% 🍏

@rndudals rndudals merged commit 2a2607e into dev Oct 3, 2024
3 checks passed
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.

2 participants