[taler-wallet-core] 02/02: console to logger

[gnunet]

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

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

commit 1a06f6700604b19da9fdef9334447f48de0341a6
Author: Sebastian <sebasjm@gmail.com>
AuthorDate: Thu Apr 21 15:39:30 2022 -0300

    console to logger
---
 .../taler-wallet-webextension/src/background.ts    |  3 ++-
 .../taler-wallet-webextension/src/wxBackend.ts     | 23 ++++++++++++----------
 2 files changed, 15 insertions(+), 11 deletions(-)

diff --git a/packages/taler-wallet-webextension/src/background.ts 
b/packages/taler-wallet-webextension/src/background.ts
index ab572ea7..134bec19 100644
--- a/packages/taler-wallet-webextension/src/background.ts
+++ b/packages/taler-wallet-webextension/src/background.ts
@@ -24,8 +24,8 @@
  * Imports.
  */
 import { platform, setupPlatform } from "./platform/api.js";
-import firefoxAPI from "./platform/firefox.js";
 import chromeAPI from "./platform/chrome.js";
+import firefoxAPI from "./platform/firefox.js";
 import { wxMain } from "./wxBackend.js";
 
 const isFirefox = typeof (window as any) !== 'undefined' && typeof (window as 
any)['InstallTrigger'] !== 'undefined'
@@ -48,6 +48,7 @@ try {
   console.error(e);
 }
 
+// setGlobalLogLevelFromString("trace")
 platform.notifyWhenAppIsReady(() => {
   wxMain();
 })
diff --git a/packages/taler-wallet-webextension/src/wxBackend.ts 
b/packages/taler-wallet-webextension/src/wxBackend.ts
index c6ab3ed8..b0c4102f 100644
--- a/packages/taler-wallet-webextension/src/wxBackend.ts
+++ b/packages/taler-wallet-webextension/src/wxBackend.ts
@@ -26,7 +26,7 @@
 import {
   classifyTalerUri,
   CoreApiResponse,
-  CoreApiResponseSuccess, TalerErrorCode,
+  CoreApiResponseSuccess, Logger, TalerErrorCode,
   TalerUriType,
   WalletDiagnostics
 } from "@gnu-taler/taler-util";
@@ -66,6 +66,8 @@ let outdatedDbVersion: number | undefined;
 
 const walletInit: OpenedPromise<void> = openPromise<void>();
 
+const logger = new Logger("wxBackend.ts");
+
 async function getDiagnostics(): Promise<WalletDiagnostics> {
   const manifestData = platform.getWalletVersion();
   const errors: string[] = [];
@@ -136,13 +138,13 @@ async function dispatch(
     }
     case "wxSetExtendedPermissions": {
       const newVal = req.payload.value;
-      console.log("new extended permissions value", newVal);
+      logger.trace("new extended permissions value", newVal);
       if (newVal) {
         platform.registerTalerHeaderListener(parseTalerUriAndRedirect);
         r = wrapResponse({ newValue: true });
       } else {
         const rem = await 
platform.getPermissionsApi().remove(getReadRequestPermissions());
-        console.log("permissions removed:", rem);
+        logger.trace("permissions removed:", rem);
         r = wrapResponse({ newVal: false });
       }
       break;
@@ -184,7 +186,7 @@ async function reinitWallet(): Promise<void> {
   try {
     currentDatabase = await openTalerDatabase(indexedDB as any, reinitWallet);
   } catch (e) {
-    console.error("could not open database", e);
+    logger.error("could not open database", e);
     walletInit.reject(e);
     return;
   }
@@ -202,12 +204,12 @@ async function reinitWallet(): Promise<void> {
     timer = new SetTimeoutTimerAPI();
   }
 
-  console.log("setting wallet");
+  logger.info("Setting up wallet");
   const wallet = await Wallet.create(currentDatabase, httpLib, timer, 
cryptoWorker);
   try {
     await wallet.handleCoreApiRequest("initWallet", "native-init", {});
   } catch (e) {
-    console.error("could not initialize wallet", e);
+    logger.error("could not initialize wallet", e);
     walletInit.reject(e);
     return;
   }
@@ -216,7 +218,7 @@ async function reinitWallet(): Promise<void> {
     platform.sendMessageToAllChannels(message)
   });
   wallet.runTaskLoop().catch((e) => {
-    console.log("error during wallet task loop", e);
+    logger.error("error during wallet task loop", e);
   });
   // Useful for debugging in the background page.
   if (typeof window !== "undefined") {
@@ -254,7 +256,7 @@ function parseTalerUriAndRedirect(tabId: number, talerUri: 
string): void {
       // handleNotifyReserve(w);
       break;
     default:
-      console.warn(
+      logger.warn(
         "Response with HTTP 402 has Taler header, but header value is not a 
taler:// URI.",
       );
       break;
@@ -268,6 +270,7 @@ function parseTalerUriAndRedirect(tabId: number, talerUri: 
string): void {
  * Sets up all event handlers and other machinery.
  */
 export async function wxMain(): Promise<void> {
+  logger.trace("starting")
   const afterWalletIsInitialized = reinitWallet();
 
   platform.registerReloadOnNewVersion();
@@ -285,14 +288,14 @@ export async function wxMain(): Promise<void> {
   try {
     platform.registerTalerHeaderListener(parseTalerUriAndRedirect);
   } catch (e) {
-    console.log(e);
+    logger.error("could not register header listener", e);
   }
 
   // On platforms that support it, also listen to external
   // modification of permissions.
   platform.getPermissionsApi().addPermissionsListener((perm, lastError) => {
     if (lastError) {
-      console.error(lastError);
+      logger.error(`there was a problem trying to get permission ${perm}`, 
lastError);
       return;
     }
     platform.registerTalerHeaderListener(parseTalerUriAndRedirect);

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