some fixes, v7.5.1

Author: tavikukko

package.json

@@ -1,6 +1,6 @@
 {
   "name": "sp-editor",
-  "version": "7.5.0",
+  "version": "7.5.1",
   "private": true,
   "homepage": ".",
   "dependencies": {

public/manifest.json

@@ -1,7 +1,7 @@
 {
   "name": "SP Editor",
   "homepage_url": "https://microsoftedge.microsoft.com/addons/detail/affnnhcbfmcbbdlcadgkdbfafigmjdkk",
-  "version": "7.5.0",
+  "version": "7.5.1",
   "description": "Create and update SharePoint Online/SP2013/SP2016/SP2019 css/js files, inject files to web, manage web/list properties, list Webhook",
   "manifest_version": 3,
   "devtools_page": "devtools.html",

src/pages/proxy/chrome/addproxy.ts

@@ -12,6 +12,16 @@ export const addProxyScript = (proxies: IProxy[], enabled: boolean, update?: boo
     }
   }
 
+  function safeParseHeaders(headers: string | undefined): Headers | undefined {
+    if (!headers) return undefined;
+    try {
+      return new Headers(JSON.parse(headers));
+    } catch (error) {
+      console.error('Failed to parse headers:', error);
+      return undefined;
+    }
+  }
+
   if (!enabled) {
     if ((window as any).__fetchInterceptorRegistered) {
       window.fetch = (window as any).__originalFetch;
@@ -44,14 +54,14 @@ export const addProxyScript = (proxies: IProxy[], enabled: boolean, update?: boo
 
       for (const proxy of proxies) {
         const methodMatches = proxy.methods.includes('ALL') || proxy.methods.includes(method);
-        if (proxy.enabled && url && url.indexOf(proxy.url) > -1 && methodMatches && shouldFail(proxy.failRate)) {
+        if (proxy.enabled && url && url.indexOf(proxy.url.trim()) > -1 && methodMatches && shouldFail(proxy.failRate)) {
           console.warn(`SP Editor blocked ${url}`);
           return new Response(
-            JSON.stringify(proxy.responseBody),
+            proxy.responseBody,
             {
               status: Number(proxy.status),
               statusText: proxy.statusText,
-              headers: proxy.responseHeaders,
+              headers: safeParseHeaders(proxy.responseHeaders),
             }
           );
         }

src/pages/proxy/chrome/chrome-actions.ts

@@ -7,20 +7,15 @@ import { addProxyScript } from './addproxy';
 let listener: (tabId: number, changeInfo: any, tab: any) => void; // Listener reference
 
 export async function addProxy(dispatch: Dispatch<ProxyActions | HomeActions>, enabled: boolean, payload: IProxy[], update?: boolean) {
-  if ((enabled && !chrome.tabs.onUpdated.hasListeners()) || (enabled && update)) {
-    if (chrome.tabs.onUpdated.hasListeners()) {
-      chrome.tabs.onUpdated.removeListener(listener);
-    }
-
+  if ((enabled)) {
+    chrome.tabs.onUpdated.removeListener(listener);
     listener = (tabId, changeInfo, tab) => {
       if (changeInfo.status === 'complete' && tab.active) {
         executeScript(dispatch, enabled, payload, false);
       }
     };
-    if(!chrome.tabs.onUpdated.hasListeners()) {
-      chrome.tabs.onUpdated.addListener(listener);
-    }
-  } else if (!enabled && chrome.tabs.onUpdated.hasListeners()) {
+    chrome.tabs.onUpdated.addListener(listener);
+  } else {
     chrome.tabs.onUpdated.removeListener(listener);
   }
 

src/pages/proxy/components/editpanel.tsx

@@ -26,12 +26,12 @@ const ProxyEditPanel = () => {
     url: '/_api/',
     status: '200',
     statusText: 'OK',
-    responseHeaders: {
+    responseHeaders: JSON.stringify({
       'Content-Type': 'application/json',
-    },
-    responseBody: {
+    }),
+    responseBody: JSON.stringify({
       message: 'Success',
-    },
+    }),
     description: '',
   });
 
@@ -74,7 +74,7 @@ const ProxyEditPanel = () => {
             }
 
             const data = await response.json();
-            setSampleData(data);
+            setSampleData(JSON.stringify(data, null, 2));
           } catch (error) {
             setSampleData(undefined);
           }
@@ -244,9 +244,9 @@ const ProxyEditPanel = () => {
           label="Response Headers"
           multiline
           autoAdjustHeight
-          value={JSON.stringify(localProxy.responseHeaders, null, 2)}
+          value={typeof localProxy.responseHeaders === 'object' ? JSON.stringify(localProxy.responseHeaders, null, 2) : localProxy.responseHeaders}
           onChange={(e, newValue) =>
-            setLocalProxy({ ...localProxy, responseHeaders: newValue ? JSON.parse(newValue) : {} })
+            setLocalProxy({ ...localProxy, responseHeaders: newValue ?? '' })
           }
           styles={{ root: { width: '100%' } }}
         />
@@ -255,9 +255,9 @@ const ProxyEditPanel = () => {
             label="Response Body"
             multiline
             autoAdjustHeight
-            value={JSON.stringify(localProxy.responseBody, null, 2)}
+            value={typeof localProxy.responseBody === 'object' ? JSON.stringify(localProxy.responseBody, null, 2) : localProxy.responseBody}
             onChange={(e, newValue) =>
-              setLocalProxy({ ...localProxy, responseBody: newValue ? JSON.parse(newValue) : {} })
+              setLocalProxy({ ...localProxy, responseBody: newValue ?? '' })  
             }
             styles={{ root: { width: '100%' } }}
           />

src/store/proxy/reducers.ts

@@ -7,11 +7,11 @@ const defaultProxy: IProxy[] = [{
   url: '/_api/',
   status: '429',
   statusText: 'Too many requests',
-  responseHeaders: {
+  responseHeaders: JSON.stringify({
     'Content-Type': 'application/json',
     'Retry-After': '2', // Tell the client to wait 2 seconds
-  },
-  responseBody: {
+  }),
+  responseBody: JSON.stringify({
     error: {
       code: '-2147024860, Microsoft.SharePoint.SPQueryThrottledException',
       message: {
@@ -20,7 +20,7 @@ const defaultProxy: IProxy[] = [{
           'The attempted operation is prohibited because it exceeds the list view threshold enforced by the administrator.',
       },
     },
-  },
+  }),
   description: 'Default 429 proxy, please edit to suit your needs',
 },
   {
@@ -30,10 +30,10 @@ const defaultProxy: IProxy[] = [{
     url: 'https://graph.microsoft.com/v1.0/me',
     status: '200',
     statusText: 'OK',
-    responseHeaders: {
+    responseHeaders: JSON.stringify({
       'Content-Type': 'application/json',
-    },
-    responseBody: {
+    }),
+    responseBody: JSON.stringify({
       "@odata.context": "https://graph.microsoft.com/v1.0/$metadata#users/$entity",
       "businessPhones": [
         "+1 412 555 0109"
@@ -48,7 +48,7 @@ const defaultProxy: IProxy[] = [{
       "surname": "Bowen",
       "userPrincipalName": "[email protected]",
       "id": "48d31887-5fad-4d73-a9f5-3c356e68a038"
-    },
+    }),
     description: 'Default 200 proxy, please edit to suit your needs',
   }
 ];

src/store/proxy/types.ts

@@ -10,8 +10,8 @@ export interface IProxy {
   url: string
   status: string
   statusText: string
-  responseHeaders: HeadersInit
-  responseBody: any
+  responseHeaders: string
+  responseBody: string
   enabled?: boolean
   description?: string
 }