What the developer wrote
Authored by julian
spark min fee error info logging
What changed, and why it matters
This commit only adds a warning log message when a Spark mint transaction's calculated fee is too low. It does not change any logic, checks, or behavior. The same exception was already being thrown before; now the app also records an explanatory message for debugging.
No security action required. Treat as routine logging improvement.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff inserts a single Logging.instance.w(…) call before an existing throw Exception(‘fee is less than vSize’). No control flow, validation thresholds, fee calculation, or error handling changed. It is purely an observability/diagnostics addition.
Changed components
lib/wallets/wallet/wallet_mixin_interfaces/spark_interface.dartInspect captured patch +3 / −0
diff --git a/lib/wallets/wallet/wallet_mixin_interfaces/spark_interface.dart b/lib/wallets/wallet/wallet_mixin_interfaces/spark_interface.dart
index 3b4e819..2fa4637 100644
--- a/lib/wallets/wallet/wallet_mixin_interfaces/spark_interface.dart
+++ b/lib/wallets/wallet/wallet_mixin_interfaces/spark_interface.dart
@@ -1994,6 +1994,9 @@ mixin SparkInterface<T extends ElectrumXCurrencyInterface>
);
if (nFeeRet.toInt() < data.vSize!) {
+ Logging.instance.w(
+ "Spark mint transaction failed: $nFeeRet is less than ${data.vSize}",
+ );
throw Exception("fee is less than vSize");
}
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.