[taler-wallet-core] branch master updated: remove excessive logging

[gnunet]

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

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

The following commit(s) were added to refs/heads/master by this push:
     new 270d84d6 remove excessive logging
270d84d6 is described below

commit 270d84d611792ee3a9764e58321f0139e60ea3ec
Author: Florian Dold <florian@dold.me>
AuthorDate: Mon Jan 11 00:22:06 2021 +0100

    remove excessive logging
---
 packages/taler-integrationtests/src/harness.ts     | 11 +++++-----
 .../src/crypto/workers/cryptoImplementation.ts     | 24 ----------------------
 .../taler-wallet-core/src/types/backupTypes.ts     |  8 +-------
 3 files changed, 7 insertions(+), 36 deletions(-)

diff --git a/packages/taler-integrationtests/src/harness.ts 
b/packages/taler-integrationtests/src/harness.ts
index ebf1e98d..58bcf2cf 100644
--- a/packages/taler-integrationtests/src/harness.ts
+++ b/packages/taler-integrationtests/src/harness.ts
@@ -1068,19 +1068,20 @@ export class ExchangeService implements 
ExchangeServiceInterface {
   }
 
   async revokeDenomination(denomPubHash: string) {
-    if (this.isRunning()) {
-      throw Error("exchange must be stopped when revoking denominations");
+    if (!this.isRunning()) {
+      throw Error("exchange must be running when revoking denominations");
     }
     await runCommand(
       this.globalState,
-      "exchange-keyup",
-      "taler-exchange-keyup",
+      "exchange-offline",
+      "taler-exchange-offline",
       [
         "-c",
         this.configFilename,
         ...this.timetravelArgArr,
-        "--revoke",
+        "revoke-denomination",
         denomPubHash,
+        "upload",
       ],
     );
   }
diff --git 
a/packages/taler-wallet-core/src/crypto/workers/cryptoImplementation.ts 
b/packages/taler-wallet-core/src/crypto/workers/cryptoImplementation.ts
index d8045f72..1f44d627 100644
--- a/packages/taler-wallet-core/src/crypto/workers/cryptoImplementation.ts
+++ b/packages/taler-wallet-core/src/crypto/workers/cryptoImplementation.ts
@@ -408,17 +408,12 @@ export class CryptoImplementation {
 
     const planchetsForGammas: RefreshPlanchet[][] = [];
 
-    logger.trace("starting RC computation");
-
     for (let i = 0; i < kappa; i++) {
       const transferKeyPair = setupRefreshTransferPub(
         decodeCrock(refreshSessionSecretSeed),
         i,
       );
       sessionHc.update(transferKeyPair.ecdhePub);
-      logger.trace(
-        `HASH transfer_pub ${encodeCrock(transferKeyPair.ecdhePub)}`,
-      );
       transferPrivs.push(encodeCrock(transferKeyPair.ecdhePriv));
       transferPubs.push(encodeCrock(transferKeyPair.ecdhePub));
     }
@@ -427,17 +422,11 @@ export class CryptoImplementation {
       for (let i = 0; i < denomSel.count; i++) {
         const r = decodeCrock(denomSel.denomPub);
         sessionHc.update(r);
-        logger.trace(`HASH new_coins ${encodeCrock(r)}`);
       }
     }
 
     sessionHc.update(decodeCrock(meltCoinPub));
-    logger.trace(`HASH coin_pub ${meltCoinPub}`);
     sessionHc.update(amountToBuffer(valueWithFee));
-    logger.trace(
-      `HASH melt_amount ${encodeCrock(amountToBuffer(valueWithFee))}`,
-    );
-
     for (let i = 0; i < kappa; i++) {
       const planchets: RefreshPlanchet[] = [];
       for (let j = 0; j < newCoinDenoms.length; j++) {
@@ -465,26 +454,13 @@ export class CryptoImplementation {
             coinEvHash: encodeCrock(hash(ev)),
           };
           planchets.push(planchet);
-
-          logger.trace(
-            `GENERATE i=${i} coin=${coinNumber} m=${encodeCrock(
-              pubHash,
-            )} bf=${encodeCrock(blindingFactor)} dp=${encodeCrock(
-              denomPub,
-            )} ev=${encodeCrock(ev)}`,
-          );
-
           sessionHc.update(ev);
-          logger.trace(`HASH ev ${encodeCrock(ev)}`);
         }
       }
       planchetsForGammas.push(planchets);
     }
 
     const sessionHash = sessionHc.finish();
-
-    logger.trace(`RHASH ${encodeCrock(sessionHash)}`);
-
     const confirmData = buildSigPS(SignaturePurpose.WALLET_COIN_MELT)
       .put(sessionHash)
       .put(decodeCrock(meltCoinDenomPubHash))
diff --git a/packages/taler-wallet-core/src/types/backupTypes.ts 
b/packages/taler-wallet-core/src/types/backupTypes.ts
index 5570902e..5e4844b5 100644
--- a/packages/taler-wallet-core/src/types/backupTypes.ts
+++ b/packages/taler-wallet-core/src/types/backupTypes.ts
@@ -524,12 +524,6 @@ export interface BackupTip {
   finish_clock?: ClockStamp;
   finish_is_failure?: boolean;
 
-  finish_info?: {
-    timestamp: Timestamp;
-    clock: ClockStamp;
-    failure: boolean;
-  };
-
   /**
    * The tipped amount.
    */
@@ -579,7 +573,7 @@ export enum BackupRefreshReason {
  */
 export interface BackupRefreshSession {
   /**
-   * Hased denominations of the newly requested coins.
+   * Hashed denominations of the newly requested coins.
    */
   new_denoms: BackupDenomSel;
 

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