return error instead of just logging it

ten-protocol · Nov 1, 2023 · eb4cbb9 · eb4cbb9
1 parent f6ac73a
commit eb4cbb9
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/tools/walletextension/accountmanager/account_manager.go b/tools/walletextension/accountmanager/account_manager.go
@@ -69,6 +69,7 @@ func (m *AccountManager) ProxyRequest(rpcReq *wecommon.RPCRequest, rpcResp *inte
 		err = m.subscriptionsManager.HandleNewSubscriptions(clients, rpcReq, rpcResp, userConn)
 		if err != nil {
 			m.logger.Error("Error subscribing to multiple clients")
+			return err
 		}
 		return err
 	}

diff --git a/tools/walletextension/wallet_extension.go b/tools/walletextension/wallet_extension.go
@@ -249,6 +249,7 @@ func (w *WalletExtension) AddAddressToUser(hexUserID string, message string, sig
 	encClient, err := common.CreateEncClient(w.hostAddr, addressFromMessage.Bytes(), privateKeyBytes, signature, w.Logger())
 	if err != nil {
 		w.Logger().Error(fmt.Errorf("error creating encrypted client for user: (%s), %w", hexUserID, err).Error())
+		return errors.New(fmt.Errorf("error creating encrypted client for user: (%s), %w", hexUserID, err).Error())
 	}
 
 	accManager.AddClient(addressFromMessage, encClient)