Merge pull request #38 from michael-peterson-cisco/FlagRemoveUnnecess…

…aryHostDots-doesn't-remove-extra-dots-between-parts-#36

purell.go

@@ -86,6 +86,7 @@ var rxDWORDHost = regexp.MustCompile(`^(\d+)((?:\.+)?(?:\:\d*)?)$`)
 var rxOctalHost = regexp.MustCompile(`^(0\d*)\.(0\d*)\.(0\d*)\.(0\d*)((?:\.+)?(?:\:\d*)?)$`)
 var rxHexHost = regexp.MustCompile(`^0x([0-9A-Fa-f]+)((?:\.+)?(?:\:\d*)?)$`)
 var rxHostDots = regexp.MustCompile(`^(.+?)(:\d+)?$`)
+var rxHostInteriorDots = regexp.MustCompile(`\.+`)
 var rxEmptyPort = regexp.MustCompile(`:+$`)
 
 // Map of flags to implementation function.
@@ -368,6 +369,7 @@ func removeUnncessaryHostDots(u *url.URL) {
 				u.Host += matches[2]
 			}
 		}
+		u.Host = rxHostInteriorDots.ReplaceAllString(u.Host, ".")
 	}
 }
 

purell_test.go

@@ -486,6 +486,13 @@ var (
 			"http://www.example.com/",
 			false,
 		},
+		{
+			"UnnecessaryHostDots-5",
+			"http://www..example...com/",
+			FlagsSafe | FlagRemoveUnnecessaryHostDots,
+			"http://www.example.com/",
+			false,
+		},
 		{
 			"EmptyPort-1",
 			"http://www.thedraymin.co.uk:/main/?p=308",