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

2단계 - 배포하기 #326

Merged
merged 7 commits into from
Mar 20, 2022
Merged

2단계 - 배포하기 #326

merged 7 commits into from
Mar 20, 2022

Conversation

ssamzag
Copy link

@ssamzag ssamzag commented Mar 19, 2022

안녕하세요.
2단계 배포 검수 요청드립니다.
설정 때문에 은근 애 먹었네요..

Copy link
Contributor

@brainbackdoor brainbackdoor left a comment

Choose a reason for hiding this comment

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

2단계 미션 잘 진행해주셨네요~
몇가지 확인할 부분이 있어 피드백 남겨요
주말 잘 보내세요~

@@ -60,7 +60,8 @@ ssamzag-nextstep-infrapem.pem
### 2단계 - 배포하기
1. TLS가 적용된 URL을 알려주세요

- URL :
- URL : https://pleasesubway.p-e.kr
Copy link
Contributor

Choose a reason for hiding this comment

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

현재 운영 디비를 internal이 아닌 public 서버에 띄워져있네요.
2단계 요구사항은 internal에 디비를 띄운후 was 가 운영디비를 바라보도록 구성하는것도 포함입니다.

Copy link
Contributor

Choose a reason for hiding this comment

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

이제 8080 포트에 대한 정책은 불필요하지 않을까요

Copy link
Author

Choose a reason for hiding this comment

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

안녕하세요.
intenal에 띄워 놓고 바라보도록 했는데 public에 띄운 것도 보셨나 봅니다..ㅠ public 디비는 내리도록 하겠습니다.
8080도 제거하고 다시 리뷰요청 드릴게요

Copy link
Contributor

Choose a reason for hiding this comment

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

  upstream app {
    server 172.17.0.1:8080;
  }

nginx.conf 설정을 변경해야 할듯해요.
현재 https://pleasesubway.p-e.kr -> 3.38.40.77:8080 으로 연결되도록 설정되어 있는데요.
그러다보니, 도메인으로 연결시 공인 IP의 8080으로 재요청을 하는데, 보안그룹에 8080 정책을 제거해서 연결이 안되네요.
nginx 연결시 was 의 8080으로 연결되도록 구성해야 하는데, 서버간 통신은 private ip 로 요청하도록 구성해야 해요.
reverse proxy 는 was 를 외부에 노출하지 않도록 하는 역할도 하고 있어요.

Copy link
Contributor

Choose a reason for hiding this comment

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

application-prod.properties 등의 코드 변경사항도 커밋에 추가하여 리뷰요청 해주세요.
서버에서 직접 코드를 변경하는게 아니라 변경사항들을 원격저장소로 관리하는 것도 요구사항이에요.

Copy link
Author

Choose a reason for hiding this comment

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

properties 분리해 놨는데 머지하면서 제가 잘 못해서 꼬였나봅니다.
그리고 서버간 통신을 private 으로 바라보도록 변경하였습니다.
감사합니다.

This was referenced Mar 19, 2022
Copy link
Contributor

@brainbackdoor brainbackdoor left a comment

Choose a reason for hiding this comment

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

피드백 잘 반영하셨네요.
추가로 확인할 부분들이 있어 피드백 남겼어요~

@@ -60,7 +60,8 @@ ssamzag-nextstep-infrapem.pem
### 2단계 - 배포하기
1. TLS가 적용된 URL을 알려주세요

- URL :
- URL : https://pleasesubway.p-e.kr
Copy link
Contributor

Choose a reason for hiding this comment

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

  upstream app {
    server 172.17.0.1:8080;
  }

nginx.conf 설정을 변경해야 할듯해요.
현재 https://pleasesubway.p-e.kr -> 3.38.40.77:8080 으로 연결되도록 설정되어 있는데요.
그러다보니, 도메인으로 연결시 공인 IP의 8080으로 재요청을 하는데, 보안그룹에 8080 정책을 제거해서 연결이 안되네요.
nginx 연결시 was 의 8080으로 연결되도록 구성해야 하는데, 서버간 통신은 private ip 로 요청하도록 구성해야 해요.
reverse proxy 는 was 를 외부에 노출하지 않도록 하는 역할도 하고 있어요.

@@ -60,7 +60,8 @@ ssamzag-nextstep-infrapem.pem
### 2단계 - 배포하기
1. TLS가 적용된 URL을 알려주세요

- URL :
- URL : https://pleasesubway.p-e.kr
Copy link
Contributor

Choose a reason for hiding this comment

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

application-prod.properties 등의 코드 변경사항도 커밋에 추가하여 리뷰요청 해주세요.
서버에서 직접 코드를 변경하는게 아니라 변경사항들을 원격저장소로 관리하는 것도 요구사항이에요.

@ssamzag
Copy link
Author

ssamzag commented Mar 20, 2022

.

Copy link
Contributor

@brainbackdoor brainbackdoor left a comment

Choose a reason for hiding this comment

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

피드백 반영 잘 해주셨네요.
2단계 미션 진행하느라 고생하셨어요~

@brainbackdoor brainbackdoor merged commit 5dd57da into next-step:ssamzag Mar 20, 2022
This pull request was closed.
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