Merge pull request #5 from ShareChat/fix/santizeUrl

sanitizeUrl regex remove semi update
ShareChat · Jan 3, 2023 · 81dc8b4 · 81dc8b4
2 parents 3f74b45 + 463e565
commit 81dc8b4
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -45,7 +45,7 @@ export const sanitizeString = (
  */
 export const sanitizeUrl = (
   str: string,
-  expression: RegExp = /[`~!$%^*()|+;'"<>{}[\]\\]/gi
+  expression: RegExp = /[`~!$%^*()|+'"<>{}[\]\\]/gi
 ) => (str ? str.replace(expression, '') : str);
 
 /**
@@ -65,7 +65,7 @@ export const sanitizeUrl = (
  */
 export const getSafeSearchParam = (
   p: string,
-  expression: RegExp = /[`~!$%^*()|+;'"<>{}[\]\\]/gi
+  expression: RegExp = /[`~!@#$%^&*()|+=?;:'",.<>{}[\]\\/]/gi
 ): string | null => {
   if (typeof window !== 'undefined') {
     const params = new URLSearchParams(window.location?.search);

diff --git a/test/sanitize.test.ts b/test/sanitize.test.ts
@@ -38,6 +38,6 @@ describe('Test Sanitize String', () => {
         search: '?answer=<script>alert("Hello")</script>',
       },
     }));
-    expect(getSafeSearchParam('answer')).toEqual('scriptalertHello/script');
+    expect(getSafeSearchParam('answer')).toEqual('scriptalertHelloscript');
   });
 });