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] 이강혁 로그인 #6

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

Conversation

hyxklee
Copy link
Member

@hyxklee hyxklee commented May 16, 2024

요구사항

✅ 회원가입
✅ 로그인 (ID 중복 확인, 비밀번호 2차 확인)
✅ 출석체크 (출석 로그 확인 및 출석률 계산)

회원가입 로직 테스트

image

회고

스프링 시큐리티를 이용하여 로그인을 구현하는 것과 개발자가 서비스 단에서 로그인을 구현하는 것 중에 고민을 많이 했습니다.

  • 스프링 시큐리티의 필터를 커스텀해서 구현하면 프레임워크를 이용하는 만큼 간단하고 효율적인 구현이 가능하지만 API를 명시적으로 매핑하긴 어려워보였습니다.
  • 서비스 단에서 로그인을 구현하는 것은 API 유지보수 측면에서 유리해보였으나 프레임워크를 사용하는 것보다 효율성이 떨어져보였습니다.
  • 결론을 내진 못했으니 계속 고민해보고, 실무에서는 어떻게 구현하는지 더 찾아보겠습니다.

hyxklee added 27 commits May 10, 2024 15:09
@hyxklee hyxklee self-assigned this May 16, 2024
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