Skip to content

docs: Apache/NGINX reverse proxy configs #6086

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

Closed
bpmct opened this issue Feb 7, 2023 · 3 comments · Fixed by #6185 or #6213
Closed

docs: Apache/NGINX reverse proxy configs #6086

bpmct opened this issue Feb 7, 2023 · 3 comments · Fixed by #6185 or #6213
Assignees

Comments

@bpmct
Copy link
Member

bpmct commented Feb 7, 2023

We get a lot of questions in Discord about this!

@bpmct bpmct self-assigned this Feb 7, 2023
@matifali
Copy link
Member

Which section of the docs should we add these?
https://coder.com/docs/v2/latest
https://coder.com/docs/v2/latest/quickstart
or maybe a new section https://coder.com/docs/v2/latest/configuration
that includes three separate files for caddy, nginx, and apache.

cc: @bpmct

@bpmct
Copy link
Member Author

bpmct commented Feb 13, 2023

@matifali Right now, let's put these in a section in https://coder.com/docs/v2/latest/configuration and then keep the files in https://github.com/coder/coder/tree/main/examples/web-server/.

@bpmct
Copy link
Member Author

bpmct commented Feb 13, 2023

We can also consider refreshing our Caddy docs in scope for this. @coadler suggested checking out headscale's doc with example configs. Coder's config may actually require similar configuration: https://github.com/juanfont/headscale/blob/main/docs/reverse-proxy.md

bpmct added a commit that referenced this issue Feb 15, 2023
* docs: Add nginx reverse-proxy example

This PR adds nginx reverse-proxy example to provision coder with tls certificate using letsencrypt certbot.

This will partially resolve #6086.

* change nginx example to to absolute path

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* refactor: replaced bullets with numbered lists

* remove the ambiguous ip addr.

* fixed a typo

* correctly handle the wildcard subdomain

* simplified after testing

* fmt: prettier formatting

* Adapt to the coder style guide

* fix: agent disconnection

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update docs/admin/configure.md

Co-authored-by: Ben Potter <me@bpmct.net>

* Update examples/web-server/nginx/README.md

Co-authored-by: Ben Potter <me@bpmct.net>

* updated with suggested changes

* updated with requested changes

* add reference to certbot docs for other dns providers

---------

Co-authored-by: Ben Potter <me@bpmct.net>
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 a pull request may close this issue.

2 participants