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

박대경 5주차 과제 제출 #16

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ggok0265
Copy link

@ggok0265 ggok0265 commented Nov 9, 2024

Question

  • 세션 기반 로그인 시스템에서 혹시나 제가 잘못 이해하여 작성한 부분이 있는지 궁금합니다

Reference

Copy link
Collaborator

@inhooo00 inhooo00 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 깔끔하게 잘 정리해 주셨습니다. 고민해보면 좋을 내용 몇 개 질문 드립니다!!

  1. 세션 기반 로그인과 쿠키 기반 로그인은 어떤 장단점이 있나요? 왜 우리는 쿠키 기반 로그인도 있는데 세션 기반 로그인으로 넘어오게 되었을까요?
  2. SecurityConfig 부분을 보면 logout을 http 메서드 안에서 진행하고 있습니다! login 로직은 controller에서 진행하고 있는데 말이죠. 두 방식 중 무엇이 좋을까요? 대경님의 의견이 궁금합니다.

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.

2 participants