환경 변수 통합

[Insung-Jo]

🗣️ 논의 주제: 환경 변수 통합

📅 작성일: 06-17

🙋‍♂️ 작성자: 조인성

---

🧩 문제 상황 / 논의 배경

현재 어떤 문제가 발생했는지, 또는 왜 이 논의를 하게 되었는지 작성해주세요.

• 과거 PR 에도 엔드 포인트 관련해서 환경변수로 관리하면 좋을 것 같다는 말을 듣기도 하였고 최근 PR에서도 각각의 환경 변수를 사용하는 부분을 확인해서 이 부분 통합을 한 번 하는 게 좋다고 판단 했습니다!

• 이유: 만약 API 호출 부분에서 해당 환경 변수가 없을 경우 작업 하는 사람마다 오류가 발생할 수 있다고 판단

예시:

// .env
NEXT_PUBLIC_API= ....


// api 부분 
api.get(process.env.NEXT_PUBLIC_API)
api.post(process.env.NEXT_PUBLIC_URL, data) // undefined, data와 같음

• 이 경우에 만약 해당된 환경 변수가 없을 경우 동작 하지 않습니다.

---

🎯 논의 목표

이 논의가 어떤 결정을 위한 것인지 명확히 적어주세요.

• 환경 변수 통일을 위함

---

📌 현재 제안된 안 또는 선택지

아래처럼 항목을 정리해주시고, 팀원들이 👍 로 투표할 수 있도록 하면 좋아요!

• 안으로 작성된 부분은 제가 현재 사용하고 있는 것을 작성한 것과 피드백을 반영한 부분을 반영한 것 입니다.
• 이 부분은 전체 의견을 확인해봐야 할 거 같아서 댓글로 작성해주세요!

1. ✅ 안 A: API URL만 명시 결정
   (저는 현재 이렇게 사용하고 있습니다.)

NEXT_PUBLIC_API_URL= https://example.com
NEXT_PUBLIC_TEAM_ID= 15-2 // 추가

2. 💡 안 B: 팀 기수 포함

NEXT_PUBLIC_API_URL= https://example.com/15-2

---

[LeeCh0129]

초기에는 15-2로 기수 포함해서 안B로 사용 했었으나 따로 인성님이 작성한 authEndPoint 코드를 보고 통일성을 위해서 안A로 저 또한 아래와 같이 사용중입니다.
NEXT_PUBLIC_API_URL= https://example.com
NEXT_PUBLIC_TEAM_ID= 15-2

따로 다른 기수의 더미 데이터를 통해서 테스트 하시는분도 계실 수 있으시니 안 A로 하고, 팀 기수의 경우도 따로 환경변수로 분리해서 사용하면 테스트하기 용이할거 같네요.

[Insung-Jo]

의견 감사합니다!

[yuj2n]

저도 임시 기수로 테스트하는 경우에는 안 A와 같이 분리하는 게 좋아보입니다

[dkslel1225]

안B가 깔끔하다 생각했는데, 찬호님 의견 보니까 테스트하려면 안 A가 좋을것 같긴 하네요

[Insung-Jo]

다들 A안이 좋을 거 같다고 하셔서 A안으로 결정하겠습니다!