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

Refactor(auth): 인증 시스템 전반 개선 및 에러 처리 강화 #8

Merged
merged 8 commits into from
Feb 18, 2025

Conversation

reach0908
Copy link
Owner

🔍️ 이 PR을 통해 해결하려는 문제가 무엇인가요?

인증(Auth) 시스템의 전반적인 개선과 리팩토링을 통해 다음과 같은 문제들을 해결하고자 합니다:

  • 토큰 기반 인증의 안정성 강화
  • 서버 컴포넌트의 이점을 최대한 활용할 수 있는 구조로 개선
  • 전역 에러 처리 시스템 구축
  • 타입 안정성 강화

✨ 이 PR에서 핵심적으로 변경된 사항은 무엇일까요?

  1. 인증 시스템 리팩토링

    • lib/auth 디렉토리 구조 개선 및 모듈화
    • TanStack Query를 활용한 데이터 페칭 최적화
    • 토큰 만료 시간 확인 로직 개선
    • 서버 컴포넌트 prefetch 도입으로 성능 최적화
  2. 에러 처리 시스템 개선

    • 전역 에러 페이지 추가 (app/error.tsx)
    • 인증 관련 에러 처리 로직 강화
  3. 타입 시스템 개선

    • 인증 관련 타입 정의 개선 및 분리
    • 타입 안정성 강화를 위한 유틸리티 함수 추가

🔖 핵심 변경 사항 외에 추가적으로 변경된 부분이 있나요?

  1. 프로젝트 구조 개선

    • Landing 페이지 컴포넌트 위치 이동 및 구조 개선
    • 프로바이더 구조 개선 및 최적화
  2. 설정 파일 업데이트

    • Next.js 설정 파일 업데이트
    • ESLint 설정 개선
    • 패키지 의존성 업데이트

🙏 Reviewer 분들이 이런 부분을 신경써서 봐 주시면 좋겠어요

  1. 서버 컴포넌트와 클라이언트 컴포넌트의 분리가 적절한지
  2. 인증 관련 타입 시스템이 충분히 안전한지
  3. 에러 처리 방식이 사용자 경험을 해치지 않는지
  4. TanStack Query 도입이 적절하게 이루어졌는지

🩺 이 PR에서 테스트 혹은 검증이 필요한 부분이 있을까요?

  1. 토큰 만료 시나리오 테스트

    • 토큰 만료 시 자동 로그아웃
    • 만료된 토큰으로 API 요청 시 에러 처리
  2. 에러 페이지 동작 검증

    • 다양한 에러 상황에서의 에러 페이지 표시
    • 에러 복구 기능 검증
  3. 인증 플로우 테스트

    • 로그인/로그아웃 기능
    • 인증이 필요한 페이지 접근 제어
    • 새로고침 시 인증 상태 유지

Copy link

linear bot commented Feb 18, 2025

@reach0908 reach0908 self-assigned this Feb 18, 2025
@reach0908 reach0908 merged commit 4755982 into develop Feb 18, 2025
3 checks passed
@reach0908 reach0908 deleted the feature/kjh-214 branch February 18, 2025 03:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant