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

Configure HTTPS #35

Merged
merged 6 commits into from
Jun 24, 2023
Merged

Configure HTTPS #35

merged 6 commits into from
Jun 24, 2023

Conversation

thompsonmj
Copy link
Contributor

Installed certs with certbot into EC2 instance, mounted volume into docker-compose, and configured nginx to use encryption. Forwards http to https.

Auto cert renewal coming in a future issue.

This branch came directly off of main, and commits to it were cherry-picked from https (which was branched from 15-react) to select only the relevant HTTPS-related changes.

Copy link
Collaborator

@johnbradley johnbradley left a comment

Choose a reason for hiding this comment

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

Could you add some documentation explaining the steps used to setup certbot?
Perhaps add this info to the https://github.com/Imageomics/Andromeda#deployment section.

@thompsonmj
Copy link
Contributor Author

README entry added

Copy link
Member

@egrace479 egrace479 left a comment

Choose a reason for hiding this comment

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

LGTM

Comment on lines +34 to +35
sudo dnf install nginx -y
sudo systemctl start nginx
Copy link
Collaborator

Choose a reason for hiding this comment

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

Is it necessary to temporarily install nginx and run it on the host VM?
What happens if you don't start nginx?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The alternative to installing nginx on the host is to configure Certbot to work with the Dockerized nginx. Either way would work fine, but if neither is done, Certbot won't be able to validate our control of the domain.

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this is ok for now, but could you create an issue to research using the nginx container we are already with certbot:

FROM nginx:stable

Installing nginx in two places could lead to confusion down the road if we need to change an nginx config setting.

Copy link
Contributor Author

@thompsonmj thompsonmj Jun 23, 2023

Choose a reason for hiding this comment

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

OK, sounds good! Now part of #37.

README.md Outdated Show resolved Hide resolved
Co-authored-by: John Bradley <johnbradley2008@gmail.com>
Copy link
Collaborator

@johnbradley johnbradley left a comment

Choose a reason for hiding this comment

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

LGTM

@thompsonmj thompsonmj merged commit 59f6bc4 into main Jun 24, 2023
1 check passed
@thompsonmj thompsonmj deleted the https-2 branch June 26, 2023 16:56
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.

None yet

3 participants