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

[Bug] recoil-persist 적용되지 않는 문제 #1244

Open
izone00 opened this issue Feb 27, 2024 · 0 comments
Open

[Bug] recoil-persist 적용되지 않는 문제 #1244

izone00 opened this issue Feb 27, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@izone00
Copy link
Contributor

izone00 commented Feb 27, 2024

Desc

  • recoil-persist가 localStorage에 상태를 기록해 둘 때 recoil_key의 난수 때문에 기록을 가져오지 못하는 상황입니다.
    이러한 이유로 localStorage에 불필요한 값이 쌓이고 있는 것 같아요

Reproduce

  1. 배포된 환경에서 확인

Wanted

  • recoil_key의 난수 제거
  • recoil-persist 제거
  • 자동 로그인 로직 수정

Bug img / video

  • 문제 상황
Screen Shot 2024-02-27 at 5 19 14 PM
  • 문제가 되는 코드
Screen Shot 2024-02-27 at 5 30 18 PM

Etc

  • recoil_key의 난수를 제거하면 recoil-persist가 정상 작동하게 되어, localStage의 recoil상태값으로 로그인 여부를 판단하게 됩니다.
  • recoil-persist가 정상 작동하면, 개발 환경에서 next가 SSR을 통해 만들어주는 html과 웹 브라우저가 그려주는 html이 충돌하는 오류가 나타나는 것 같습니다. (react hydration)
  • 자동 로그인 시에 recoil-persist로 로그인 여부를 확인하는 방식의 수정이 필요해 보입니다.
  • [Refactoring] cookie (refresh token) 보안문제 #1197 와 함께 해결해야 할 것 같습니다.
@izone00 izone00 added the bug Something isn't working label Feb 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant