What the developer wrote
Authored by f321x
qml: change event callback logs to debug
Change per-tx event callback logs to debug, might make the process
a bit less clogged if there are many txs.
What changed, and why it matters
This commit simply changes two logging messages from 'info' level to 'debug' level in the mobile/QML wallet interface. It reduces routine log output when many transactions occur. There is no security-relevant change.
No security action needed. Treat as routine logging/UX cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
In electrum/gui/qml/qewallet.py, two @qt_event_listener callbacks (on_event_new_transaction and on_event_adb_tx_height_changed) now log at debug instead of info. The functional behavior (notification, model dirty flags) is unchanged. No code paths, permissions, or data handling are modified.
Changed components
electrum/gui/qml/qewallet.pyInspect captured patch +2 / −2
diff --git a/electrum/gui/qml/qewallet.py b/electrum/gui/qml/qewallet.py
index 0842e79..6b0cc9e 100644
--- a/electrum/gui/qml/qewallet.py
+++ b/electrum/gui/qml/qewallet.py
@@ -200,7 +200,7 @@ class QEWallet(AuthMixin, QObject, QtEventListener):
@qt_event_listener
def on_event_new_transaction(self, wallet: 'Abstract_Wallet', tx: Transaction):
if wallet == self.wallet:
- self._logger.info(f'new transaction {tx.txid()}')
+ self._logger.debug(f'new transaction {tx.txid()}')
self.add_tx_notification(tx)
if self._addressCoinModel is not None: # only setDirty if it was already initialized
self._addressCoinModel.setDirty()
@@ -213,7 +213,7 @@ class QEWallet(AuthMixin, QObject, QtEventListener):
@qt_event_listener
def on_event_adb_tx_height_changed(self, adb, txid, old_height, new_height):
if adb == self.wallet.adb:
- self._logger.info(f'tx_height_changed {txid}. {old_height} -> {new_height}')
+ self._logger.debug(f'tx_height_changed {txid}. {old_height} -> {new_height}')
self.historyModel.setDirty() # assuming wallet.is_up_to_date triggers after
@qt_event_listener
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.