Closed Ipswitch closed 2 years ago
First of all, thank you for your contribution. While I understand the idea behind the change, unfortunately we can't merge this.
Of course, you oftentimes want to have firewall rules and restrictions like this. But, the terraform modules in this repo are meant to be for evaluation and demonstration purposes only. As such they should work in all kinds of environments.
This change would create multiple problems:
opendns.com
to be available. While it is unlikely to go away, I don't think adding such a dependency to a terraform module is a good idea.
Limits access to the instance to reduce attack surface area. Adds result of whatsismyip.sh to the TF. Requires jq to be installed.
Increased instance storage size.