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

/api 라우팅을 path rewrite가 아닌 application level에서 추가 #446

Open
2 tasks
SnowSuno opened this issue Jan 24, 2024 · 0 comments
Open
2 tasks
Labels
d-needed (Discussion Needed) 의논 필요 ⚒ enhancement New feature or request

Comments

@SnowSuno
Copy link
Member

Summary

현재에는 백엔드 서버에서는 / 레벨에서 리스닝하고 있고, /api path를 붙여주는 것은 nginx 단에서 담당하고 있지만, express의 root router 단에서 /api를 붙이는 방식을 제안합니다.

sparcs-kaist/taxi-front#704 에서 로컬 네트워크 앱 개발 환경 구성을 위해 dev proxy를 사용하고 있는데, 신경 써주어야 할 부분이 많아져서 아예 정석적인(?) 방법으로 바꾸는 것은 어떨지 제안합니다.
prod 환경의 nginx config도 훨씬 간단해질 것으로 예상됩니다.

Tasks

  • express root router에 /api prefix 적용하기
  • nginx config에서 path rewrite 규칙 삭제하기
@SnowSuno SnowSuno added ⚒ enhancement New feature or request d-needed (Discussion Needed) 의논 필요 labels Jan 24, 2024
@SnowSuno SnowSuno changed the title /api 라우팅을 path rewrite가 아닌 리스너 단에서 추가 /api 라우팅을 path rewrite가 아닌 application level에서 추가 Jan 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
d-needed (Discussion Needed) 의논 필요 ⚒ enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant