fix: open ports 80/443 in proxy module when proxies are configured #5

Merged
greg merged 1 commits from klaatu/nixos:fix/proxy-firewall into main 2026-03-25 22:37:43 +00:00
Collaborator

The greg.proxies module enables nginx and creates virtualHost entries, but never opened the firewall for the ports nginx listens on. This means every proxied service on every host (Jellyfin, Grafana, and anything else using greg.proxies) has been silently unreachable from the LAN since the proxy module was introduced.

Fix: add allowedTCPPorts = [80 443] to the proxy module, conditioned on the same mkIf ((attrValues cfg) != []) guard already used for the nginx config block.

Opened by Klaatu 🌌

The `greg.proxies` module enables nginx and creates virtualHost entries, but never opened the firewall for the ports nginx listens on. This means every proxied service on every host (Jellyfin, Grafana, and anything else using `greg.proxies`) has been silently unreachable from the LAN since the proxy module was introduced. Fix: add `allowedTCPPorts = [80 443]` to the proxy module, conditioned on the same `mkIf ((attrValues cfg) != [])` guard already used for the nginx config block. > Opened by Klaatu 🌌
klaatu added 1 commit 2026-03-25 22:36:03 +00:00
fix: open ports 80/443 in proxy module when proxies are configured
buildbot/nix-eval Build done. (1 warning)
buildbot/nix-build Build done.
a5719c4e02
The greg.proxies module enables nginx but never opened the firewall,
so all proxied services (Jellyfin, Grafana, etc.) were unreachable
on the LAN. Add allowedTCPPorts [80 443] conditioned on the same
mkIf guard as the nginx virtualHosts config.
greg merged commit 68fdf7c37c into main 2026-03-25 22:37:43 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: greg/nixos#5