Skip to content

feature: added caddy for easier setup and exposing #40

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

dhruva71
Copy link

Adds caddy to the docker compose; adds a caddy folder with a Caddyfile for easier setup. This makes it easy to expose over the internet, and has automatic HTTPS support.

Copy link
Member

@netroy netroy left a comment

Choose a reason for hiding this comment

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

I understand why you'd want this addition to the repo, but plenty of folks are using nginx and traefik instead.
If we make a change like this, we need to include those services as well, and somehow use only one of the three via profiles.

@dhruva71
Copy link
Author

Understood. I'll work on it to add support for those two as far as possible. My initial view was ease of setup, but I hadn't considered that viewpoint. Thanks!

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