fix: 어드민 로그아웃 세션 정리#528
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
워크스루이 PR은 관리자 로그아웃 흐름을 안전성과 사용자 경험 측면에서 개선합니다. 1. 세션 재발급 중 로그아웃 시 발생하는 경합 상태 방지 2. 백엔드 로그아웃 API 엔드포인트 추가 3. AdminLayout 로그아웃 핸들러를 비동기 흐름으로 변경 4. 로그인 페이지에서 loggedOut 쿼리 파라미터 처리 예상 코드 리뷰 난이도🎯 3 (Moderate) | ⏱️ ~20 분 제안 리뷰어
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
ESLint skipped: no ESLint configuration detected in root package.json. To enable, add 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. Comment |
관련 이슈
작업 내용
/auth/sign-outAPI를 호출하도록 연결했습니다./auth/reissue가 다시 성공해 세션이 복구되는 흐름을 막기 위해 로그인 페이지의 로그아웃 직후 자동 세션 확인을 건너뜁니다.특이 사항
v23.10.0이라 루트engines.node: 22.x경고가 출력되지만 검증은 통과했습니다.리뷰 요구사항 (선택)
/auth/login?loggedOut=1에서 reissue를 건너뛰는 방식이 운영 플로우에 맞는지 확인 부탁드립니다.검증
pnpm --filter @solid-connect/admin lint:checkpnpm --filter @solid-connect/admin typecheckpnpm --filter @solid-connect/admin build@solid-connect/admin ci:check@solid-connect/admin ci:check,@solid-connect/admin build