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: 환경 변수를 zod로 파싱 #530

Merged
merged 23 commits into from
Jun 25, 2023
Merged

Conversation

scarf005
Copy link
Member

@scarf005 scarf005 commented Jun 24, 2023

개요

#485 프로젝트의 일부입니다.

작업 사항

  • 모든 process.env를 zod schema로 파싱 및 검증했습니다.
  • src/config를 단일 진리의 원천으로 하여 여러 곳에서 중복된 키를 정의한 것을 줄였습니다.
  • process.env를 그대로 사용하지 않고 사용하기 적합한 설정 객체로 후처리한 것을 사용하게 해 편의성을 올렸습니다.
  • 파싱한 설정 타입들을 tsdoc으로 문서화했습니다.

목적

env키가 존재하지 않거나 올바르지 않을 경우 (예: REDIRECT_URL이 URL이 아님) 런타임에 문제가 발생하거나, 조용히 잊혀져 문제를 찾는 데 어려움을 겪을 수 있습니다.

@scarf005 scarf005 marked this pull request as ready for review June 24, 2023 12:33
Copy link
Member

@YeonSeong-Lee YeonSeong-Lee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@scarf005
Copy link
Member Author

image

키값 업데이트 후 인증이 잘 되는 지 확인해 보고 머지하도록 하겠습니다.

@scarf005
Copy link
Member Author

일시적인 현상이었던 것 같습니다. PR 이전과 비교해 정상 동작하는 것을 확인했습니다.

@scarf005 scarf005 merged commit 0a32ceb into develop Jun 25, 2023
@scarf005 scarf005 deleted the env-모두-zod로-파싱 branch June 25, 2023 08:37
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.

2 participants