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

[20기_이채원] Docker를 이용한 배포 미션 제출합니다. #59

Open
wants to merge 7 commits into
base: gchaewon
Choose a base branch
from

Conversation

gchaewon
Copy link

No description provided.

Copy link

@nimikgnoej nimikgnoej left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!
성공적으로 과제를 마무리 하셨네요 🐳

Comment on lines +2852 to +2854
```bash
vi .env
```

Choose a reason for hiding this comment

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

코드를 보니 docker-compose.yml 에서도 환경변수들을 넣어주고 계시던데 두 방법 모두를 쓴 것 일까요?

Copy link
Author

Choose a reason for hiding this comment

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

앗 저 도커 이미지를 컴포즈 말고 dockerfile로 생성해서 env 파일을 별도로 넘겨주는 방식으로 했습니다! 컴포즈 파일 변경한건 저번 세션에서 다른 분들 코드보고 참고해서 수정해보았어요 ^__^

Comment on lines +2889 to +2893
`http://{publicIp}:80/{테스트할URI}`

이렇게 요청을 보내면 된다.

회원가입과 로그인 기능이 잘 동작하는 것을 확인할 수 있다.

Choose a reason for hiding this comment

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

깰끔하게 성공

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