Auth/#62

[seonghunYang]

📌 작업 내용

구현 내용 및 작업 했던 내역

• 인증 절차 로직을 개발하고, 동작을 검증하기 위해 모킹했습니다.
• 인증 여부에 따른 페이지 라우팅을 미들웨어로 구현했습니다.
• 로그인 성공 시 마이페이지로 리다이렉트하는 기능으로 동작을 변경했습니다.

🤔 고민 했던 부분

• 개발한 부분은 Next middleware에서 인증 여부에 따른 페이지 라우팅을 하는 부분이므로, 테스트 방법이 애매하여 테스트 코드를 작성하지 못했습니다. 이런 상황을 테스트하려면 cypress와 같은 e2e 수준의 테스트 도구가 필요해 보입니다.
• sign-in 성공 시, useRouter로 리다이렉트 할지, redirect() 함수로 리다이렉트 할지 두 가지 선택지가 있었습니다. useRouter는 클라이언트 사이드에서 라우팅이기 때문에 아키텍처 상으로 굳이 한 단계를 거칠 필요가 없다고 판단하여 redirect()를 사용했습니다.
  • 참고 문헌: https://nextjs.org/docs/app/building-your-application/routing/redirecting
• fetch 시 토큰이 필요한 경우, 매번 반복적으로 헤더에 넣어줘야 하는 불편함이 있습니다. 이를 해결하기 위해 프로젝트 초반에 논의했던 fetch를 wrapping하는 작업이 필요하며, 만약 이 작업을 수행한다면 지금이 적절한 타이밍일 것 같습니다. fetch wrapping 시에는 헤더 설정과 에러 처리 설정 기능 등이 필요해 보입니다.
• 사용자 정보를 통해 PDF 파일 제출 여부를 판단해야 하는데, 아직 API가 명확히 정의되지 않아 임시로 처리하였습니다. API가 확정되면 이를 수정할 예정입니다.
• 개발 편의성을 위해, searchParam에 ?isAuth=true일 때만 미들웨어가 작동하도록 설정했습니다. 필요할 때 사용하고, 그 외의 시간에는 사용하지 않아도 됩니다.

🔊 도움이 필요한 부분

[github-actions]

🚀storybook: https://65ccb85d5afe55a024495bc0-hjljnwevms.chromatic.com/

[github-actions]

🚀storybook: https://65ccb85d5afe55a024495bc0-abpvtawkuq.chromatic.com/

[gahyuun]

고생하셨습니다!

[gahyuun]

혹시 isAuth = true의 의도에 대해 조금 더 설명해주실 수 있나요??

[seonghunYang]

• 미들웨어의 역할은 토큰 기반 유효성 검사를 수행하고, 검사가 통과된 경우에만 해당 페이지에 접근이 가능하게 하는 것입니다.
• 그러나 이 경우, 인증이 필요한 페이지 개발이 불편할 수 있으므로, 개발 단계에서는 해당 조건문으로 동작하지 않도록 설정하였습니다.
• 만약 인증에 따른 페이지 라우팅 동작이 필요하다면, isAuth=true로 searchParam을 설정하시면 됩니다.

[gahyuun]

accessToken이 존재하면 validateToken 함수를 호출하네요!
accessToken이 존재하는 경우마다 새로 토큰을 재발급하여 요청을 보내는 플로우 같은데, 이 이유가 있나용??
저는 accessToken이 존재하면 해당 accessToken으로 요청을 보내고, error 응답시 토큰 재발급 후 재요청 이 플로우를 생각했어서 여쭤봅니다!

[seonghunYang]

• 말씀하신대로 일반 fetch 요청의 경우 그렇게 처리할 수 있습니다. 실제로, 클라이언트 사이드에서 인증이 실패하면(401 에러 발생 시) 에러 바운더리에서 이를 처리하는 로직을 다음 작업으로 개발할 계획입니다.
• 그러나 이 경우는 페이지 라우팅에 대한 권한 인증이며, 현재 Next와 API 서버를 분리하고 있는 저희 아키텍처 구조상, 해당 페이지에 접근이 가능한지 검사하기 위해서는 해당 토큰이 유효한지를 검사할 수밖에 없습니다.
• 이유는 Next 서버와 API 서버가 분리되어 있기 때문입니다. SSR을 위한 Next 서버에는 인증 정보가 없으므로, API 서버에서 이 요청이 유효한지 확인한 후에 라우팅을 수행할 수 있습니다.

[gahyuun]

넵 확인했습니다!

[yougyung]

어제 회의에 의하면 isSumbitted해당 컬럼은 서버로부터 안내려오는 것 아닌가요?

[seonghunYang]

• 현재 해당 API에 대한 명세가 정확하게 되어 있지 않아서 일단 임의로 처리하였습니다.
• 추후 명세가 확정되면 변경하도록 하겠습니다.