diff --git a/README.md b/README.md index 47a9495..033d9bd 100644 --- a/README.md +++ b/README.md @@ -404,9 +404,10 @@ address nginx passes on and no other. The image: - Listens on port 8080 by default (override with `PORT` env var) - Takes the client address from `X-Forwarded-For` only on requests from the - reverse proxies named in `TRUSTED_PROXIES`, and by default from none. - `bin/entrypoint.sh` checks each entry with `netwatch-server check-cidr`, which - parses it as the backend parses its own `TRUSTED_PROXIES` + reverse proxies named in `TRUSTED_PROXIES`, by default from any address in + `10.0.0.0/8`, `172.16.0.0/12` or `192.168.0.0/16`. `bin/entrypoint.sh` checks + each entry with `netwatch-server check-cidr`, which parses it as the backend + parses its own `TRUSTED_PROXIES` - Sends access logs to stdout - Caches static assets with immutable headers - Sends the security headers `REPO_POLICIES.md` requires on every response, as @@ -425,9 +426,9 @@ What the [upaas](https://git.eeqj.de/sneak/upaas) app for netwatch needs: - **Port:** container port `8080`. - **Volume:** container path `/data`; the reports are kept in `/data/reports`. -- **Environment variables:** none is required. An empty one counts as unset, and - one set to a value netwatch cannot use stops the container at start, with the - reason in its log. +- **Environment variables:** none is required. An empty one counts as unset, + except `TRUSTED_PROXIES`, and one set to a value netwatch cannot use stops the + container at start, with the reason in its log. - `PORT`, default `8080`: the container port, from 1 to 65535. `8081` cannot be used: the backend listens on it inside the container - `REPORTS_PER_MINUTE`, default `60`: reports each client address may send a @@ -441,16 +442,17 @@ What the [upaas](https://git.eeqj.de/sneak/upaas) app for netwatch needs: in: `/data` or a path below it, with no `.` or `..` part and no extra `/`. The container also stops if the path goes through a symbolic link that leads out of `/data` or is written as a full path - - `TRUSTED_PROXIES`, default empty: set it to the address the reverse proxy - in front of the container connects from, as an IP address or CIDR; several - are separated by commas. nginx takes the client address from - `X-Forwarded-For` only on a request from one of them, and the rate limit - counts that address. Unset, `X-Forwarded-For` is ignored and every client - behind the proxy shares the proxy's one allowance of `REPORTS_PER_MINUTE`. - Name only addresses nothing but the proxy connects from: any client that - connects from one can write its own `X-Forwarded-For`, and through a port - Docker publishes, every client may connect from the Docker network's - gateway, such as `172.17.0.1`. + - `TRUSTED_PROXIES`, default `10.0.0.0/8,172.16.0.0/12,192.168.0.0/16`: the + addresses the reverse proxy in front of the container connects from, as IP + addresses or CIDRs separated by commas. nginx takes the client address + from `X-Forwarded-For` only on a request from one of them, and the rate + limit counts that address. A list replaces the default. Set and empty, + `X-Forwarded-For` is ignored and every client behind the proxy shares the + proxy's one allowance of `REPORTS_PER_MINUTE`. Any client that connects + from one of these addresses can write its own `X-Forwarded-For`, and + through a port Docker publishes, every client may connect from the Docker + network's gateway, such as `172.17.0.1`, which the default includes; where + that matters, name only the addresses nothing but the proxy connects from. - `METRICS_USERNAME` and `METRICS_PASSWORD`, default empty: with both set, the backend records Prometheus metrics of its requests and serves them at `/metrics` on the container port, to requests with this user name and diff --git a/TODO.md b/TODO.md index 76023ff..816e938 100644 --- a/TODO.md +++ b/TODO.md @@ -21,6 +21,12 @@ Run `make frontend-viewport-test` in CI as its own step; it is not part of # Completed Steps +- 2026-10-14: with `TRUSTED_PROXIES` unset, the container's nginx trusts the + three private ranges of RFC 1918, `10.0.0.0/8`, `172.16.0.0/12` and + `192.168.0.0/16` ([#119](https://git.eeqj.de/sneak/netwatch/issues/119)), as + sneak ruled on 2026-09-22 for every app; it trusted no proxy before. A list + replaces them, and set and empty it trusts none. The backend inside the + container still trusts only `127.0.0.1/32` - 2026-10-14: the repo has the layout `REPO_POLICIES.md` gives ([#30](https://git.eeqj.de/sneak/netwatch/issues/30)). The Go module is at the root (`go.mod`, `go.sum`, `.golangci.yml`, `cmd/netwatch-server/`, diff --git a/bin/entrypoint.sh b/bin/entrypoint.sh index ceea002..8bf4ec4 100755 --- a/bin/entrypoint.sh +++ b/bin/entrypoint.sh @@ -34,9 +34,11 @@ fi # TRUSTED_PROXIES names the reverse proxies in front of the container, # as IP addresses or CIDRs separated by commas. nginx takes the client -# address from X-Forwarded-For only on a request from one of them, so -# unset or empty, it trusts no one. configs/nginx.conf includes the file -# written here, one set_real_ip_from line per entry. +# address from X-Forwarded-For only on a request from one of them. +# Unset, they are the three private ranges of RFC 1918; set and empty, +# nginx trusts no one, which is why the default below follows - and not +# :-. configs/nginx.conf includes the file written here, one +# set_real_ip_from line per entry. # # nginx looks up an entry it cannot read as an address as a hostname, # and trusts what it finds (1.2.3 is found as 1.2.0.3). So each entry @@ -45,7 +47,7 @@ fi # TRUSTED_PROXIES. Its error, naming the CIDR, is dropped for the one # below, naming the entry as written. set -f keeps a * in an entry from # becoming a list of file names. -TRUSTED_PROXIES="${TRUSTED_PROXIES:-}" +TRUSTED_PROXIES="${TRUSTED_PROXIES-10.0.0.0/8,172.16.0.0/12,192.168.0.0/16}" set -f for proxy in $(printf '%s' "$TRUSTED_PROXIES" | tr ',' ' '); do case "$proxy" in diff --git a/configs/nginx.conf b/configs/nginx.conf index 53f821c..f04e425 100644 --- a/configs/nginx.conf +++ b/configs/nginx.conf @@ -18,9 +18,10 @@ server { # The client address comes from X-Forwarded-For only on a request # from the reverse proxies in TRUSTED_PROXIES: bin/entrypoint.sh - # writes one set_real_ip_from line for each into this file, and - # leaves it empty when TRUSTED_PROXIES is unset, so that by default - # the client address is the one each request comes from. + # writes one set_real_ip_from line for each into this file. Unset, + # TRUSTED_PROXIES is the three private ranges of RFC 1918; set and + # empty, the file is empty and the client address is the one each + # request comes from. include /etc/nginx/trusted-proxies.conf; real_ip_header X-Forwarded-For; real_ip_recursive on; diff --git a/script/frontend-viewport-test b/script/frontend-viewport-test index fba6fb8..42977a1 100755 --- a/script/frontend-viewport-test +++ b/script/frontend-viewport-test @@ -64,8 +64,8 @@ main() { # configs/nginx.conf is a template: the image renders it over its own # default.conf, with the same port and limit bin/entrypoint.sh uses. # The empty file it includes trusts no proxy, as bin/entrypoint.sh - # writes it when TRUSTED_PROXIES is unset. It also includes the - # security headers, so the page runs under the shipped policy. + # writes it when TRUSTED_PROXIES is set and empty. It also includes + # the security headers, so the page runs under the shipped policy. docker run -d --rm --name "$SERVER" \ --network "$NETWORK" --network-alias netwatch \ -e PORT=8080 -e NGINX_ENVSUBST_FILTER='^PORT$' \