gnunet-svn
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[taler-wallet-core] branch master updated: remove clipboardRead permissi


From: gnunet
Subject: [taler-wallet-core] branch master updated: remove clipboardRead permission & pretty
Date: Thu, 22 Dec 2022 16:14:07 +0100

This is an automated email from the git hooks/post-receive script.

sebasjm pushed a commit to branch master
in repository wallet-core.

The following commit(s) were added to refs/heads/master by this push:
     new f1962dceb remove clipboardRead permission & pretty
f1962dceb is described below

commit f1962dceb3e7cfbe3615f9611866b54fb9f5c4e6
Author: Sebastian <sebasjm@gmail.com>
AuthorDate: Thu Dec 22 12:13:59 2022 -0300

    remove clipboardRead permission & pretty
---
 .../taler-wallet-webextension/manifest-v2.json     |  3 +-
 .../taler-wallet-webextension/manifest-v3.json     |  3 +-
 .../src/platform/chrome.ts                         | 77 ++++++++++++----------
 .../src/wallet/Settings.tsx                        |  8 +--
 packages/taler-wallet-webextension/src/wxApi.ts    |  6 +-
 .../taler-wallet-webextension/src/wxBackend.ts     |  6 +-
 6 files changed, 53 insertions(+), 50 deletions(-)

diff --git a/packages/taler-wallet-webextension/manifest-v2.json 
b/packages/taler-wallet-webextension/manifest-v2.json
index 74f103feb..8c336da25 100644
--- a/packages/taler-wallet-webextension/manifest-v2.json
+++ b/packages/taler-wallet-webextension/manifest-v2.json
@@ -24,8 +24,7 @@
   "optional_permissions": [
     "http://*/*";,
     "https://*/*";,
-    "webRequest",
-    "clipboardRead"
+    "webRequest"
   ],
   "browser_action": {
     "default_icon": {
diff --git a/packages/taler-wallet-webextension/manifest-v3.json 
b/packages/taler-wallet-webextension/manifest-v3.json
index 78dcac623..fe9b75375 100644
--- a/packages/taler-wallet-webextension/manifest-v3.json
+++ b/packages/taler-wallet-webextension/manifest-v3.json
@@ -27,8 +27,7 @@
     }
   },
   "optional_permissions": [
-    "webRequest",
-    "clipboardRead"
+    "webRequest"
   ],
   "host_permissions": [
     "http://*/*";,
diff --git a/packages/taler-wallet-webextension/src/platform/chrome.ts 
b/packages/taler-wallet-webextension/src/platform/chrome.ts
index f951685d2..b7b0c6640 100644
--- a/packages/taler-wallet-webextension/src/platform/chrome.ts
+++ b/packages/taler-wallet-webextension/src/platform/chrome.ts
@@ -83,13 +83,14 @@ const hostPermissions = {
 
 export function containsClipboardPermissions(): Promise<boolean> {
   return new Promise((res, rej) => {
-    chrome.permissions.contains({ permissions: ["clipboardRead"] }, (resp) => {
-      const le = chrome.runtime.lastError?.message;
-      if (le) {
-        rej(le);
-      }
-      res(resp);
-    });
+    res(false);
+    // chrome.permissions.contains({ permissions: ["clipboardRead"] }, (resp) 
=> {
+    //   const le = chrome.runtime.lastError?.message;
+    //   if (le) {
+    //     rej(le);
+    //   }
+    //   res(resp);
+    // });
   });
 }
 
@@ -107,13 +108,14 @@ export function containsHostPermissions(): 
Promise<boolean> {
 
 export async function requestClipboardPermissions(): Promise<boolean> {
   return new Promise((res, rej) => {
-    chrome.permissions.request({ permissions: ["clipboardRead"] }, (resp) => {
-      const le = chrome.runtime.lastError?.message;
-      if (le) {
-        rej(le);
-      }
-      res(resp);
-    });
+    res(false);
+    // chrome.permissions.request({ permissions: ["clipboardRead"] }, (resp) 
=> {
+    //   const le = chrome.runtime.lastError?.message;
+    //   if (le) {
+    //     rej(le);
+    //   }
+    //   res(resp);
+    // });
   });
 }
 
@@ -187,13 +189,14 @@ export async function removeHostPermissions(): 
Promise<boolean> {
 
 export function removeClipboardPermissions(): Promise<boolean> {
   return new Promise((res, rej) => {
-    chrome.permissions.remove({ permissions: ["clipboardRead"] }, (resp) => {
-      const le = chrome.runtime.lastError?.message;
-      if (le) {
-        rej(le);
-      }
-      res(resp);
-    });
+    res(true);
+    // chrome.permissions.remove({ permissions: ["clipboardRead"] }, (resp) => 
{
+    //   const le = chrome.runtime.lastError?.message;
+    //   if (le) {
+    //     rej(le);
+    //   }
+    //   res(resp);
+    // });
   });
 }
 
@@ -775,21 +778,23 @@ async function timeout(ms: number): Promise<void> {
   return new Promise((resolve) => setTimeout(resolve, ms));
 }
 async function findTalerUriInClipboard(): Promise<string | undefined> {
-  try {
-    //It looks like clipboard promise does not return, so we need a timeout
-    const textInClipboard = await Promise.any([
-      timeout(100),
-      window.navigator.clipboard.readText(),
-    ]);
-    if (!textInClipboard) return;
-    return textInClipboard.startsWith("taler://") ||
-      textInClipboard.startsWith("taler+http://";)
-      ? textInClipboard
-      : undefined;
-  } catch (e) {
-    logger.error("could not read clipboard", e);
-    return undefined;
-  }
+  //FIXME: add clipboard feature
+  // try {
+  //   //It looks like clipboard promise does not return, so we need a timeout
+  //   const textInClipboard = await Promise.any([
+  //     timeout(100),
+  //     window.navigator.clipboard.readText(),
+  //   ]);
+  //   if (!textInClipboard) return;
+  //   return textInClipboard.startsWith("taler://") ||
+  //     textInClipboard.startsWith("taler+http://";)
+  //     ? textInClipboard
+  //     : undefined;
+  // } catch (e) {
+  //   logger.error("could not read clipboard", e);
+  //   return undefined;
+  // }
+  return undefined;
 }
 
 async function findTalerUriInActiveTab(): Promise<string | undefined> {
diff --git a/packages/taler-wallet-webextension/src/wallet/Settings.tsx 
b/packages/taler-wallet-webextension/src/wallet/Settings.tsx
index f00b242d6..c9b2205c5 100644
--- a/packages/taler-wallet-webextension/src/wallet/Settings.tsx
+++ b/packages/taler-wallet-webextension/src/wallet/Settings.tsx
@@ -115,12 +115,12 @@ export function SettingsView({
             error={autoOpenToggle.button.error.errorDetail}
           />
         )}
-        {clipboardToggle.button.error && (
+        {/* {clipboardToggle.button.error && (
           <ErrorTalerOperation
             title={<i18n.Translate>Could not toggle clipboard</i18n.Translate>}
             error={clipboardToggle.button.error.errorDetail}
           />
-        )}
+        )} */}
         <SubTitle>
           <i18n.Translate>Navigator</i18n.Translate>
         </SubTitle>
@@ -140,7 +140,7 @@ export function SettingsView({
           enabled={autoOpenToggle.value!}
           onToggle={autoOpenToggle.button.onClick!}
         />
-        <Checkbox
+        {/* <Checkbox
           label={
             <i18n.Translate>
               Automatically check clipboard for Taler URI
@@ -155,7 +155,7 @@ export function SettingsView({
           }
           enabled={clipboardToggle.value!}
           onToggle={clipboardToggle.button.onClick!}
-        />
+        /> */}
 
         <SubTitle>
           <i18n.Translate>Trust</i18n.Translate>
diff --git a/packages/taler-wallet-webextension/src/wxApi.ts 
b/packages/taler-wallet-webextension/src/wxApi.ts
index 4ec51fc78..0d33719be 100644
--- a/packages/taler-wallet-webextension/src/wxApi.ts
+++ b/packages/taler-wallet-webextension/src/wxApi.ts
@@ -58,9 +58,9 @@ export interface BackgroundOperations {
     response: void;
   };
   sum: {
-    request: number[],
-    response: number,
-  }
+    request: number[];
+    response: number;
+  };
   resetDb: {
     request: void;
     response: void;
diff --git a/packages/taler-wallet-webextension/src/wxBackend.ts 
b/packages/taler-wallet-webextension/src/wxBackend.ts
index d54dc818f..dca239de6 100644
--- a/packages/taler-wallet-webextension/src/wxBackend.ts
+++ b/packages/taler-wallet-webextension/src/wxBackend.ts
@@ -164,12 +164,12 @@ async function toggleHeaderListener(
 
 function freeze(time: number): Promise<void> {
   return new Promise((res, rej) => {
-    setTimeout(res, time)
-  })
+    setTimeout(res, time);
+  });
 }
 
 async function sum(ns: Array<number>): Promise<number> {
-  return ns.reduce((prev, cur) => prev + cur, 0)
+  return ns.reduce((prev, cur) => prev + cur, 0);
 }
 
 const backendHandlers: BackendHandlerType = {

-- 
To stop receiving notification emails like this one, please contact
gnunet@gnunet.org.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]