nix-community / terraform-nixos

A set of Terraform modules that are designed to deploy NixOS [maintainer=@adrian-gierakowski]
Apache License 2.0
333 stars 61 forks source link

deploy_nixos: change default target_port to 21 #52

Closed asymmetric closed 3 years ago

asymmetric commented 3 years ago

I found it surprising that, by not choosing a port, I wasn't ending up with the /standard/ SSH port, but with an arbitrarily chosen default.

I think defaulting to 21 is more intuitive.

grahamc commented 3 years ago

But port 22 is the standard SSH port? Port 21 is the standard port for FTP :)

asymmetric commented 3 years ago

Hah! Sorry, must be having a slow day today :joy:

grahamc commented 3 years ago

No worries :)