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

feat: 리다이렉트문을 수정합니다 #114

Merged
merged 7 commits into from
Mar 3, 2024
Merged

Conversation

geongyu09
Copy link
Collaborator

주요 변경사항

  • 기존에 page.tsx에서 처리하던 리다이렉트문을 middleware에서 처리하도록 수정하였습니다.
  • 로그인 방식이 쿠키방식으로 변경됨에 따라 로그인 관련 로직을 수정하였습니다.
    • 더이상 로그인 시 localstroage에 토큰을 저장하지 않습니다.
    • axios로직에서 withCredentials 를 추가하였습니다.
    • 더이상 로그인 여부를 확인하기 위해 를 사용하지 않습니다.
    • 로그인 여부는 서버로부터 받은 에러 코드로 확인합니다.

리뷰어에게...

  • 추가적으로 리팩토링 할 수 있는 부분이 있을까요..?
  • 현재 로그인이 안된 상태로 홈("/")에 들어가면 로그인 여부를 확인하지 않습니다(서버로 요청을 보내지 않기 때문). 그래서 로그인 페이지로 이동시키지 않는데, 기존처럼 홈("/")에서 로그인 여부를 확인하고 안되어있다면 로그인 페이지로 이동시키도록 하는 방식으로 변경하는 것이 좋을까요?

관련 이슈

closes #8 #113

@geongyu09 geongyu09 self-assigned this Mar 2, 2024
@loopy-lim
Copy link
Collaborator

리뷰하기 전에 코멘트를 한개 남기자면, 현재 로그인 방식은 약간 기믹입니다.
원래라면 로그인 검증해서 (로그인, 회원가입 / 로그아웃)을 구현하긴 합니다.

@bada308
Copy link
Collaborator

bada308 commented Mar 2, 2024

빌드 시 prettier 규칙에 맞지 않은 부분이 있어 에러가 발생합니다!

@2yunseong
Copy link
Collaborator

빌드 시 prettier 규칙에 맞지 않은 부분이 있어 에러가 발생합니다!

이거 제 잘못같네요 ㅠ 제 PR이 CI 적용이 안되서 ㅠ

Copy link
Collaborator

@bada308 bada308 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

Copy link
Collaborator

@baegyeong baegyeong left a comment

Choose a reason for hiding this comment

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

좋아요~ 수고하셨습니다.

Copy link
Collaborator

@2yunseong 2yunseong left a comment

Choose a reason for hiding this comment

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

더이상 로그인 여부를 확인하기 위해 를 사용하지 않습니다.

빠진건 Validate 컴포넌트 겠죠??!

현재 로그인이 안된 상태로 홈("/")에 들어가면 로그인 여부를 확인하지 않습니다(서버로 요청을 보내지 않기 때문). 그래서 로그인 페이지로 이동시키지 않는데, 기존처럼 홈("/")에서 로그인 여부를 확인하고 안되어있다면 로그인 페이지로 이동시키도록 하는 방식으로 변경하는 것이 좋을까요?

홈에서는 로그인을 하지 않고 갈 수 있는 페이지가 있으므로 따로 로그인을 막지 않아도 될 것 같다는 제 개인적인 생각입니다~
더군다나 그 페이지가 신청페이지이므로 어드민 부분만 막는게 괜찮다는 생각 입니다!

@geongyu09 geongyu09 merged commit d7d91a0 into main Mar 3, 2024
1 check passed
@2yunseong 2yunseong deleted the feat/8-switch-cookie branch September 9, 2024 05:56
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.

[FE] refactor: redirect는 Next의 기술을 활용한다.
5 participants