In the current state, if the user enters HTTPS (all caps like the documentation states), the module will fail to recognize it since it is comparing against https (lowercase) and will override the setting to http. If this happens, the user will have to realize what's going on and change it to https (lowercase) and then it works as expected.
With this change, the protocol is always converted to lowercase (if it is defined) so there are no issues comparing values.
In the current state, if the user enters
HTTPS
(all caps like the documentation states), the module will fail to recognize it since it is comparing againsthttps
(lowercase) and will override the setting tohttp
. If this happens, the user will have to realize what's going on and change it tohttps
(lowercase) and then it works as expected.With this change, the protocol is always converted to lowercase (if it is defined) so there are no issues comparing values.