Skip to content

dev: remove nginx

Warren Gifford requested to merge k/dev-rm-nginx into main

Created by: keegancsmith

We already have Caddy running for HTTPS support. We can use just Caddy to remove the need to have nginx in our dev environments. This does mean we won't be using the same reverse proxy as we do in production and for customers. However, I do not think that changes much in practice.

This was inspired by a conversation I had with @mrnugget. Not completely sold on the idea though after implementing the change (especially since we have caddy -> webpack -> caddy -> frontend now).

@felixfbecker is this still an issue for you? https://github.com/sourcegraph/sourcegraph/pull/10021

Merge request reports

Loading