diff options
author | pennae <github@quasiparticle.net> | 2021-12-12 03:18:04 +0100 |
---|---|---|
committer | Kerstin <kerstin@erictapen.name> | 2021-12-17 21:30:52 +0100 |
commit | 64bbe2884339dfc2e24427017b0083e73bd6181f (patch) | |
tree | 5e081bffbb76864b2e26c0f1de879c90b86662a4 /nixos/modules/services/networking/unifi.nix | |
parent | 2000a1edcd0e02a114042081632ed87b4cc55031 (diff) |
nixos/unifi: rename openPorts to openFirewall
openFirewall is the much more common name for an option with this effect. since the default was `true` all along, renaming it doesn't hurt much and only improves consistency with other modules.
Diffstat (limited to 'nixos/modules/services/networking/unifi.nix')
-rw-r--r-- | nixos/modules/services/networking/unifi.nix | 9 |
1 files changed, 5 insertions, 4 deletions
diff --git a/nixos/modules/services/networking/unifi.nix b/nixos/modules/services/networking/unifi.nix index 9750231e60342..e1908cf44d148 100644 --- a/nixos/modules/services/networking/unifi.nix +++ b/nixos/modules/services/networking/unifi.nix @@ -50,7 +50,7 @@ in ''; }; - services.unifi.openPorts = mkOption { + services.unifi.openFirewall = mkOption { type = types.bool; default = true; description = '' @@ -87,8 +87,8 @@ in config = mkIf cfg.enable { warnings = optional - (options.services.unifi.openPorts.highestPrio >= (mkOptionDefault null).priority) - "The current services.unifi.openPorts = true default is deprecated and will change to false in 22.11. Set it explicitly to silence this warning."; + (options.services.unifi.openFirewall.highestPrio >= (mkOptionDefault null).priority) + "The current services.unifi.openFirewall = true default is deprecated and will change to false in 22.11. Set it explicitly to silence this warning."; users.users.unifi = { isSystemUser = true; @@ -98,7 +98,7 @@ in }; users.groups.unifi = {}; - networking.firewall = mkIf cfg.openPorts { + networking.firewall = mkIf cfg.openFirewall { # https://help.ubnt.com/hc/en-us/articles/218506997 allowedTCPPorts = [ 8080 # Port for UAP to inform controller. @@ -196,6 +196,7 @@ in }; imports = [ (mkRemovedOptionModule [ "services" "unifi" "dataDir" ] "You should move contents of dataDir to /var/lib/unifi/data" ) + (mkRenamedOptionModule [ "services" "unifi" "openPorts" ] [ "services" "unifi" "openFirewall" ]) ]; meta.maintainers = with lib.maintainers; [ erictapen pennae ]; |