Fixed issue, using location.href resulted in possible unwanted mixin …

…with query string and get parameters when solving AMD autoload module URL
genius257 · Sep 3, 2022 · 110c114 · 110c114
1 parent 916a25a
commit 110c114
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/www/js/AMD/autoload.js b/www/js/AMD/autoload.js
@@ -55,7 +55,8 @@
                 try {
                     new URL(url);
                 } catch (e) {
-                    url = new URL(url, window.location.href).href; //FIXME: we should not use window.location.href, but only protocol, hostname and path. Query string and get parameters should be ignored.
+                    var location = window.location;
+                    url = new URL(url, `${location.protocol}//${location.hostname}${location.pathname}`).href;
                 }
                 var url = new URL(dependency, url).href;//requires us to know the url we base this call on.
                 const fileName = url.split('/').pop();