cmd/syncthing: Localhost header comparison should be case insensitive

This commit is contained in:
Jakob Borg 2016-10-03 17:34:13 +09:00
parent 891ff383ec
commit d1d565e58b
2 changed files with 3 additions and 1 deletions

View File

@ -1328,7 +1328,7 @@ func addressIsLocalhost(addr string) bool {
// There was no port, so we assume the address was just a hostname // There was no port, so we assume the address was just a hostname
host = addr host = addr
} }
switch host { switch strings.ToLower(host) {
case "127.0.0.1", "::1", "localhost": case "127.0.0.1", "::1", "localhost":
return true return true
default: default:

View File

@ -833,9 +833,11 @@ func TestAddressIsLocalhost(t *testing.T) {
}{ }{
// These are all valid localhost addresses // These are all valid localhost addresses
{"localhost", true}, {"localhost", true},
{"LOCALHOST", true},
{"::1", true}, {"::1", true},
{"127.0.0.1", true}, {"127.0.0.1", true},
{"localhost:8080", true}, {"localhost:8080", true},
{"LOCALHOST:8000", true},
{"[::1]:8080", true}, {"[::1]:8080", true},
{"127.0.0.1:8080", true}, {"127.0.0.1:8080", true},