What the developer wrote
Authored by David Adegoke
Remove print (#2764)
What changed, and why it matters
This commit removes a single debug print statement that logged a wallet object to the console. It is a cleanup change with no security relevance. There is no vulnerability, no exploit path, and no user-facing behavior change beyond preventing a wallet object from appearing in logs.
No security action required. Treat as routine code hygiene.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff deletes one line: print(sendVM.wallet); inside an onConnectDevice callback in exchange_trade_page.dart. The removed statement would have emitted the string representation of the wallet object to standard output/debug logs. Removing it reduces log noise and potential accidental sensitive-data exposure in debug builds, but it does not fix a vulnerability, change access controls, or alter application logic. The subsequent call to sendVM.hardwareWalletViewModel!.initWallet(sendVM.wallet) remains unchanged.
Changed components
lib/src/screens/exchange_trade/exchange_trade_page.dartInspect captured patch +0 / −1
diff --git a/lib/src/screens/exchange_trade/exchange_trade_page.dart b/lib/src/screens/exchange_trade/exchange_trade_page.dart
index e07d10cc..0f068094 100644
--- a/lib/src/screens/exchange_trade/exchange_trade_page.dart
+++ b/lib/src/screens/exchange_trade/exchange_trade_page.dart
@@ -216,7 +216,6 @@ class ExchangeTradeState extends State<ExchangeTradeForm> {
hardwareWalletType:
sendVM.wallet.walletInfo.hardwareWalletType!,
onConnectDevice: (context, _) {
- print(sendVM.wallet);
sendVM.hardwareWalletViewModel!.initWallet(sendVM.wallet);
Navigator.of(context).pop();
},
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.