Fixed URLSearchParams spaces (x sign) bug

See also https://gitlab.com/ClearURLs/ClearUrls/-/merge_requests/108

Author: KevinRoebert

clearurls.js

@@ -152,7 +152,7 @@ function removeFieldsFormURL(provider, pureUrl, quiet = false, request = null) {
 
         let finalURL = domain;
 
-        if (fields.toString() !== "") finalURL += "?" + fields.toString();
+        if (fields.toString() !== "") finalURL += "?" + urlSearchParamsToString(fields);
         if (fragments.toString() !== "") finalURL += "#" + fragments.toString();
 
         url = finalURL.replace(new RegExp("\\?&"), "?").replace(new RegExp("#&"), "#");

core_js/tools.js

@@ -337,3 +337,21 @@ async function sha256(message) {
 function randomASCII(len) {
     return [...Array(len)].map(() => (~~(Math.random() * 36)).toString(36)).join('');
 }
+
+/**
+ * Returns an URLSearchParams as string.
+ * Does handle spaces correctly.
+ */
+function urlSearchParamsToString(searchParams) {
+    const rtn = []
+
+    searchParams.forEach((value, key) => {
+        if (value) {
+            rtn.push(key + '=' + value)
+        } else {
+            rtn.push(key)
+        }
+    })
+
+    return rtn.join('&')
+}