-
Notifications
You must be signed in to change notification settings - Fork 3.8k
Description
Currently there are 52 services shipped by default that are using HTTP URLs by default (vs. 18 that use HTTPS):
grep -ril "http://" | wc -l
52
grep -ril "https://" | wc -l
18
As noted in another issue by @Ansuel this is inconsistent and potentially an issue regarding dependencies (wget / curl with SSL/TLS support and ca-certificates are needed, which makes the required flash size bigger).
However, default settings are important ("tyranny of defaults") and shipping insecure default settings is a bad idea.
My suggestion is as follows:
1.) Require SSL support for ddns-scripts.
2.) Re-evaluate all default settings and check if HTTPS endpoints are available. If so, change the URL to the HTTPS endpoint. If not, drop the service, so it's not shipped by default.
If someone still wants to use HTTP, the URL can still be set manually, but at least we would not configure it by default.
Optionally / alternatively a new (meta) package could be created (e.g. ddns-scripts-nossl) that will pull in all of the non-https services. In this case, however, a warning should be shown (at least in LUCI) that credetials are being sent over an insecure channel.
Obviously this is a change that will affect some users, so let's discuss this first.
I'm happy to work on this and provide a pull request, etc., but would like to first get an alignment and/or start a discussion.