Bug 1502462 - The altSrv checker does not build a uri correctly if origin is ipv6 address. r=valentin

AltSvcMapping::ProcessHeader should use GetAsciiHost instead of GetHost.

Differential Revision: https://phabricator.services.mozilla.com/D13066

--HG--
extra : moz-landing-system : lando

netwerk/protocol/http/AlternateServices.cpp

@@ -942,7 +942,17 @@ AltSvcCache::UpdateAltServiceMapping(AltSvcMapping *map, nsProxyInfo *pi,
     }
   } else {
     // for http:// resources we fetch .well-known too
-    nsAutoCString origin (NS_LITERAL_CSTRING("http://") + map->OriginHost());
+    nsAutoCString origin (NS_LITERAL_CSTRING("http://"));
+
+    // Check whether origin is an ipv6 address. In that case we need to add
+    // '[]'.
+    if (map->OriginHost().FindChar(':') != kNotFound) {
+      origin.Append('[');
+      origin.Append(map->OriginHost());
+      origin.Append(']');
+    } else {
+      origin.Append(map->OriginHost());
+    }
     if (map->OriginPort() != NS_HTTP_DEFAULT_PORT) {
       origin.Append(':');
       origin.AppendInt(map->OriginPort());

netwerk/protocol/http/nsHttpChannel.cpp

@@ -2169,7 +2169,7 @@ nsHttpChannel::ProcessAltService()
     nsAutoCString originHost;
     int32_t originPort = 80;
     mURI->GetPort(&originPort);
-    if (NS_FAILED(mURI->GetHost(originHost))) {
+    if (NS_FAILED(mURI->GetAsciiHost(originHost))) {
         return;
     }