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

/login/check API의 응답 형식 변경 #1040

Merged
merged 1 commit into from
Jan 20, 2025
Merged

Conversation

zeus6768
Copy link
Contributor

@zeus6768 zeus6768 commented Jan 18, 2025

⚡️ 관련 이슈

close #1038

작업 배경 : 프론트엔드의 요청

현재 프론트는 한번 로그인하면 name, memberId를 localStorage에 저장하는데

  1. auth token을 이제 localStorage에 보관해서 로그인 유지하려고 합니다. 이 token을 프론트에서 저장하면 name과 memberId는 이제 저장 안해도 될 것 같아요. 처음 서비스 진입 시 '/login/check' 요청을 하고 이 응답에 name, memberId를 받아오고 이는 localStorage가 아닌 프론트 코드 내부 변수로 갖고 있어도 될듯.
  2. name, memberId가 localStorage에 있느냐로 로그인 여부 판단했기에 땜에 그동안 여러 계정으로 로그인하거나 그럴때 이슈가 많기도 했음,,

📍주요 변경 사항

As-is

응답 본문 없이 상태 코드 200으로만 로그인 여부를 판단

HTTP/1.1 200

To-be

응답 본문에 사용자의 id와 이름을 포함

HTTP/1.1 200

{
  "id": 1,
  "name": "만두"
}

추가 작업

사용하지 않는 HttpServletRequest 타입의 파라미터도 삭제했어요.

🎸기타

이름, 닉네임, 사용자명... 용어 통일하고 싶군요

🍗 PR 첫 리뷰 마감 기한

01/21 18:00

@zeus6768 zeus6768 added documentation 문서 작업 BE 백엔드 labels Jan 18, 2025
@zeus6768 zeus6768 added this to the 7차 스프린트 💭 milestone Jan 18, 2025
@zeus6768 zeus6768 self-assigned this Jan 18, 2025
@zeus6768 zeus6768 linked an issue Jan 18, 2025 that may be closed by this pull request
2 tasks
@zeus6768 zeus6768 changed the title feat(auth): /login/check api의 응답 형식 변경 /login/check api의 응답 형식 변경 Jan 18, 2025
@zeus6768 zeus6768 changed the title /login/check api의 응답 형식 변경 /login/check API의 응답 형식 변경 Jan 18, 2025
@zeus6768
Copy link
Contributor Author

슬랙 알림 actions 왜 실패하죠...?

Copy link
Contributor

@zangsu zangsu left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!!!
+) 작업 배경을 Issue 본문에 수정해 두었어요. 내용 수정이 필요한 경우 얼마든지 추가적으로 수정해 주셔도 좋아요!!

@zeus6768 zeus6768 changed the title /login/check API의 응답 형식 변경 /login/check API의 응답 형식 변경 Jan 19, 2025
@zeus6768 zeus6768 closed this Jan 19, 2025
@zeus6768 zeus6768 reopened this Jan 19, 2025
@zeus6768 zeus6768 merged commit f27c4fa into dev/be Jan 20, 2025
3 of 5 checks passed
@zeus6768 zeus6768 deleted the feat/1038-login-check branch January 20, 2025 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드 documentation 문서 작업
Projects
Status: Weekend Done
Development

Successfully merging this pull request may close these issues.

[FEAT] /login/check API에서 응답으로 name, memberId 주도록 변경
2 participants