Comment

MarshallWace · May 3, 2022 · e552983 · e552983
1 parent 89569b2
commit e552983
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/spnego.go b/spnego.go
@@ -40,6 +40,11 @@ func getHostname(req *http.Request, canonicalize bool) (string, error) {
 	if req.Host != "" {
 		h = req.Host
 	}
+	// I can't think why you'd want to canonicalize an overridden Host
+	// header (you're likely to get the hostname in the URL, which is what
+	// you're overriding in the first place). But since it can be disabled,
+	// leave it here in the flow so that Host header hostnames can be
+	// canonicalized if desired.
 	if canonicalize {
 		if h, err = canonicalizeHostname(h); err != nil {
 			return "", err