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

prod, dev Infra 추가 설정 #348

Closed
3 of 4 tasks
jpham005 opened this issue Oct 4, 2023 · 3 comments · Fixed by #350
Closed
3 of 4 tasks

prod, dev Infra 추가 설정 #348

jpham005 opened this issue Oct 4, 2023 · 3 comments · Fixed by #350
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@jpham005
Copy link
Member

jpham005 commented Oct 4, 2023

프론트와 협의 후 몇몇 설정을 진행할 계획입니다.

  • nginx 설정
  • backend port 변경
  • localhost 용도의 oauth2 client 추가
  • prod AWS port 변경
@jpham005 jpham005 added this to 42Stat Oct 4, 2023
@jpham005 jpham005 converted this from a draft issue Oct 4, 2023
@jpham005 jpham005 added the enhancement New feature or request label Oct 4, 2023
@yoopark yoopark moved this from 📋 Backlog to 🔖 Ready in 42Stat Oct 4, 2023
@jpham005
Copy link
Member Author

jpham005 commented Oct 6, 2023

port 의 경우 4000 으로 변경하기로 했고, 이후 prod 에도 AWS 설정 적용해야 합니다.

@jpham005 jpham005 moved this from 🔖 Ready to 🏗 In progress in 42Stat Oct 6, 2023
@jpham005 jpham005 self-assigned this Oct 6, 2023
@jpham005 jpham005 added this to the v0.8.0 milestone Oct 6, 2023
@jpham005
Copy link
Member Author

jpham005 commented Oct 6, 2023

nginx

  • 80 을 443 redirect
  • nginx conf 의 server 삭제
  • proxy path localhost:4000 으로 변경

@jpham005 jpham005 changed the title dev 서버 설정 prod, dev Infra 추가 설정 Oct 6, 2023
@jpham005
Copy link
Member Author

jpham005 commented Oct 7, 2023

env 파일들을 전부 불러오도록 변경, 같은 값을 중복해서 작성할 필요가 없어짐.
https://docs.nestjs.com/techniques/configuration#custom-env-file-path

jpham005 added a commit that referenced this issue Oct 7, 2023
프론트와 논의 한 결과, 8080 보다 4000 이 나을 것으로 판단되어 다시 변경했습니다

- #348
@jpham005 jpham005 linked a pull request Oct 7, 2023 that will close this issue
jpham005 added a commit that referenced this issue Oct 18, 2023
프론트와 논의 한 결과, 8080 보다 4000 이 나을 것으로 판단되어 다시 변경했습니다

- #348
@github-project-automation github-project-automation bot moved this from 🏗 In progress to ✅ Done in 42Stat Oct 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

1 participant