If there are closed/terminated wallets in the system, the redemption process takes a long time. This is what happens:

1. The findWalletForRedemption function must iterate over all wallets to check which one can handle the redemption:

   tbtc-v2/typescript/src/services/redemptions/redemptions-service.ts

   Line 127 in 51094e4

   for (const wallet of wallets) {

2. To do the check, the above function must call wallets to fetch data of the particular wallet:

   tbtc-v2/typescript/src/lib/ethereum/bridge.ts

   Line 583 in 51094e4

   async wallets(walletPublicKeyHash: Hex): Promise<Wallet> {

3. The wallets function calls getWalletCompressedPublicKey under the hood:

   tbtc-v2/typescript/src/lib/ethereum/bridge.ts

   Line 607 in 51094e4

   walletPublicKey: await this.getWalletCompressedPublicKey(ecdsaWalletID),

4. Closed/terminated wallets are no longer in the contract state so, getWalletCompressedPublicKey throws while calling getWalletPublicKey:

   tbtc-v2/typescript/src/lib/ethereum/bridge.ts

   Line 521 in 51094e4

   const uncompressedPublicKey = await walletRegistry.getWalletPublicKey(

5. The above is not a problem per se as the error is caught. However, the getWalletPublicKey uses a backoff retry mechanism inside so the final throw is made after 3 internal retries which takes time and contributes to the long execution time of the entire process:

   tbtc-v2/typescript/src/lib/ethereum/wallet-registry.ts

   Line 69 in 51094e4

   const publicKey = await backoffRetrier<string>(this._totalRetryAttempts)(

The solution is simple. The backoffRetrier function should not retry if the contract call reverts with Wallet with the given ID has not been registered error. This can be achieved by passing a custom errorMatcher argument to this specific backoffRetrier call: