Fix issue 949, URLs with params

JudahGabriel · JudahGabriel · commit 4ba8ab5264ad · 2020-07-20T17:03:03.000-07:00
diff --git a/lib/manifestTools/manifestLoader.js b/lib/manifestTools/manifestLoader.js
@@ -50,7 +50,8 @@ function fetchManifestUrlFromSite (siteUrl, callback) {
     if (manifestUrl) {
       var parsedManifestUrl = url.parse(manifestUrl);
       if (!parsedManifestUrl.host) {
-        manifestUrl = url.resolve(siteUrl, parsedManifestUrl.pathname);
+        var searchOrEmpty = parsedManifestUrl.search || "";
+        manifestUrl = url.resolve(siteUrl, parsedManifestUrl.pathname + searchOrEmpty);
       }
     }
 

Original file line number	Diff line number	Diff line change
`@@ -50,7 +50,8 @@ function fetchManifestUrlFromSite (siteUrl, callback) {`
`50`	`50`	`if (manifestUrl) {`
`51`	`51`	`var parsedManifestUrl = url.parse(manifestUrl);`
`52`	`52`	`if (!parsedManifestUrl.host) {`
`53`		`- manifestUrl = url.resolve(siteUrl, parsedManifestUrl.pathname);`
	`53`	`+ var searchOrEmpty = parsedManifestUrl.search \|\| "";`
	`54`	`+ manifestUrl = url.resolve(siteUrl, parsedManifestUrl.pathname + searchOrEmpty);`
`54`	`55`	`}`
`55`	`56`	`}`
`56`	`57`