Skip to content

로그인 후 세션 갱신 경쟁 상태를 수정합니다. - #80

Merged
ghandhitechnology merged 1 commit into
mainfrom
fix/portal-sign-in
Sep 10, 2026
Merged

로그인 후 세션 갱신 경쟁 상태를 수정합니다.#80
ghandhitechnology merged 1 commit into
mainfrom
fix/portal-sign-in

Conversation

@ghandhitechnology

@ghandhitechnology ghandhitechnology commented Sep 10, 2026

Copy link
Copy Markdown
Owner

로그인 API가 성공해도 로그인 전에 시작한 세션 조회를 재사용하면 홈은 계속 로그아웃 상태를 표시하고 보호된 페이지는 로그인으로 되돌아갑니다. 세션을 새로 조회하고 가장 최근 결과가 반영된 뒤 이동하도록 수정했습니다.

  • 문제: SessionProvider의 전역 진행 중 요청을 재사용해 로그인 전 쿠키로 보낸 응답이 로그인 이후에도 적용됐습니다. 조회가 겹치면 오래된 응답이나 오류가 최신 상태를 덮을 수도 있었습니다.
  • 수정: 새로고침마다 새 세션 요청을 보내고, provider 안에서 가장 최근 요청만 세션·오류·로딩·캐시 상태를 갱신합니다. 이전 요청을 기다리던 로그인 흐름도 최신 요청 결과를 기다립니다.

변경은 client/src/components/portal/SessionProvider.tsx 한 파일에 한정했습니다. 로그인·가입·재인증 화면의 기존 세션 갱신 호출에 적용됩니다.

검증:

  • GitHub 통합 검증 게이트 통과: Prisma·migration·전체 lint·타입 검사·클라이언트 테스트·프로덕션 빌드·realtime 빌드·브리지 테스트.
  • Node.js 22에서 전체 클라이언트 테스트 143개, 타입 검사, 변경 파일 ESLint 통과.
  • 실제 LoginPageSessionProvider를 사용한 로컬 브라우저 검사에서 수정 전의 인증 성공 후 로그아웃 상태 유지 문제를 재현했습니다. 테스트 API와 합성 계정만 사용했습니다.
  • 수정 후에는 이전 응답이 먼저 또는 나중에 도착하는 경우, 세 번째 조회가 겹치는 경우, 이전 오류·현재 오류·재시도 모두 기대한 세션 상태와 이동 순서를 확인했습니다.

운영 확인: 현재 배포된 로그인 JS에는 main에 이미 병합된 로그인 후 세션 갱신도 반영되지 않았습니다. 웹 배포가 필요하며 이 환경에는 지정된 VPS 배포 키가 없어 아직 배포하지 않았습니다.

@coderabbitai

coderabbitai Bot commented Sep 10, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: c19d4ef8-36ca-46bd-9c9f-d9994d7c3fd7


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@ghandhitechnology
ghandhitechnology merged commit 6d0a257 into main Sep 10, 2026
2 checks passed
@ghandhitechnology
ghandhitechnology deleted the fix/portal-sign-in branch September 10, 2026 16:09
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