EL
← All projectsElectrum

Electrum

Long-running lightweight Bitcoin wallet for desktop and mobile platforms.

BitcoinSoftware walletsNormal
Repository coverage

1151 commits in the local evidence base

Every captured commit receives deterministic security triage and a separate communication-quality score. Security candidates and broader second-pass signals receive full-patch Ollama analysis.

92security candidates259second-pass queue1005AI analyses
131commits · 30 days
260commits · 60 days
543commits · 180 days
1000commits · 365 days
Backfill bands
Aug 5 → Feb 6470 seen11 candidatesComplete
Feb 6 → Jun 6346 seen10 candidatesComplete
Jun 6 → Jul 653 seen2 candidatesComplete
Jul 6 → Aug 554 seen3 candidatesComplete
Commit communication

Does the history explain itself?

Message quality measures whether a commit identifies its scope, purpose, rationale, testing, and supporting references. It does not change the security-severity score.

66/100 average clarity
220Strong · 80–100
548Adequate · 60–79
339Thin · 40–59
44Opaque · 0–39
2security candidates with opaque commit messaging
Read the scoring rubric →
Developer activity

Who is changing the project?

Public Git author strings; identities are not independently verified.

DeveloperCommitsCandidatesAnalyzedHigh riskMessage avg.
ThomasV137891162
SomberNight39330362063
f321x40225386070
Sander van Grieken10910101057
ghost4341516070
Felix25410072
user323062
ekzyis222053
accumulator613071
Roman Zeyde515053
Sasha Zykov515075
Aaron Fiore414068
Analysis record

Published AI watches

Last scanned 59 minutes ago

Low 29 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10982 from spesmilo/wizard_2fa_seed_redirect

This commit changes Electrum's wallet creation wizard so that if a user tries to restore a normal 'standard' wallet but pastes a seed phrase that actually belongs to a two-factor authentication (2FA) wallet, the wizard automatically redire…

UX-level safety fix preventing wallet-type/seed mismatchState mutation: wizard_data['wallet_type'] changed from 'standard' to '2fa' based on seed typeNo cryptographic, network, or privilege changes
310c9dd0by Felix+102−133 files
No security note in commit
Informational 15 AI analysisMessage 70 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

tests: wizard: cover standard -> 2fa redirection

This commit only adds and reorganizes automated test code for Electrum's wallet creation wizard. It does not change any production wallet logic, cryptography, or network handling. The tests verify that when a user tries to restore a 'stand…

70b4d27cby f321x+68−91 file
No security note in commit
Informational 15 AI analysisMessage 78 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10988 from f321x/update_security_review_ci_model

This commit updates Electrum's internal CI (continuous integration) script that runs an automated security review using Anthropic's Claude Code tool. It changes the AI model version used for reviews from 'claude-opus-5' to 'claude-opus-5-5…

CI hardening: detects and reports AI model downgrades during automated security reviewNo changes to application code, cryptography, network protocol, or build artifactsNo privilege escalation, injection, or data-exfiltration vectors introduced by the diff
638fbba8by Felix+75−111 file
No security note in commit
Informational 12 AI analysisMessage 90 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

ci: security review: warn if the model got downgraded

This commit updates Electrum's own CI security-review script. It adds detection and warnings when the automated code reviewer (Claude Code) silently falls back to a different AI model, for example after a content-policy refusal. It does no…

CI-only changeNo modification of wallet, crypto, networking, or build artifactsAdds detection for AI model fallback/downgrade in automated security review
ace2ca7aby f321x+74−101 file
No security note in commit
Low 44 AI analysisMessage 68 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10987 from spesmilo/fix_10986

This commit fixes a bug where Electrum failed to recognize certain already-signed Bitcoin transactions as complete. Specifically, for native SegWit inputs, Bitcoin Core can produce a finalized PSBT with a valid witness but an empty scriptS…

Logic error in transaction completeness detectionNative SegWit witness handling edge casePSBT interoperability issue with Bitcoin Core
bef80421by ghost43+11−02 files
No security note in commit
Low 35 AI analysisMessage 81 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

transaction: treat native segwit input with non-empty witness as complete

This commit fixes a bug where Electrum incorrectly treated finalized Bitcoin transactions as incomplete. Some wallet software (like Bitcoin Core and Sparrow) creates native SegWit transactions that omit an empty placeholder field Electrum …

Logic error in transaction completeness detectionPotential denial of service / user funds stuck due to refusal to broadcast valid finalized transactionInteroperability failure with Bitcoin Core and Sparrow PSBT output
e9bb42e6by ThomasV+11−02 files
No security note in commit
Moderate 68 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10981 from spesmilo/do_not_sign_txin_with_witness

This change prevents Electrum from re-signing Bitcoin transaction inputs that already contain a witness (the data proving the input was authorized). The patch fixes a bug where Electrum could incorrectly sign a non-SegWit input as if it we…

Incorrect signature algorithm selection for non-SegWit inputs when witness data is presentPSBT handling edge case where pre-existing witness data influences signing pathPotential invalid signature production during transaction signing
9da4c342by ThomasV+10−21 file
No security note in commit
Informational 15 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10974 from SomberNight/202609_nostr_psbt_xpub

This commit only adds a long code comment explaining a design choice in Electrum's PSBT-over-Nostr plugin. It does not change any program behavior. The comment documents that the plugin derives each cosigner's Nostr secret key from their B…

Documentation-only changeExplicit design tradeoff disclosure: cosigners can derive each other's Nostr secret keys from shared xpubsNo functional code change
46408a78by ThomasV+13−01 file
No security note in commit
Low 42 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10980 from spesmilo/add_input_from_network_do_not_ignore_exceptions

This is a one-line bugfix in Electrum's transaction handling. Previously, the code accidentally swallowed (hid) almost all exceptions when fetching extra transaction data from the network, only re-raising errors that were both network-rela…

Exception swallowing bug fixedNetwork/transaction input enrichment logic changedSilent failure mode removed
bb9d3280by ghost43+1−11 file
No security note in commit
Moderate 60 AI analysisMessage 58 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10970 from f321x/fix_bolt11_bugs_2

This commit fixes several bugs in Electrum's handling of BOLT11 Lightning invoices. The most user-visible fixes are: stricter validation of invoice amounts (rejecting zero, negative, sub-millisatoshi, and leading-zero amounts), correct pad…

Stricter BOLT11 amount validation prevents acceptance of zero, negative, sub-millisatoshi, and leading-zero amountsFixed int_to_data5 padding bug that corrupted small timestamp values during invoice round-tripMalformed 'r' routing tags now raise exceptions instead of being silently skipped
4b92c0f2by ghost43+82−939 files
No security note in commit
Moderate 61 AI analysisMessage 80 · Strong
EL ElectrumElectrum BitcoinSoftware wallets

add_info_from_network: do not swallow exceptions

Electrum's transaction builder had a bug where it would ignore certain consistency errors when fetching extra transaction data from the network. A malicious or misbehaving server could supply a wrong input amount, and Electrum would silent…

Swallowed exception leading to use of attacker-controlled input valuePSBT input consistency check bypassedPotential malicious Electrum server influence on transaction signing
3f67b21bby ThomasV+1−11 file
Vendor flagged security relevance
Informational 15 AI analysisMessage 60 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

bolt11: remove tag arg from get_routing_info

This is a small code cleanup change. A function that reads routing hints from Lightning invoices no longer takes a tag argument because only one type of routing hint ('r') is now supported. All callers are updated accordingly. There is no …

e78fba28by f321x+12−127 files
No security note in commit
Moderate 62 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10940 from accumulator/fix_10937

This commit tightens how Electrum parses Bitcoin payment requests (BOLT11 lightning invoices and BIP21 URIs). It turns previously uncaught internal errors into proper validation failures, rejects malformed invoice fields that used to be si…

Stricter input validation for externally supplied BOLT11 invoices and BIP21 URIsPreviously uncaught exceptions (ValueError, UnicodeDecodeError, ecc errors) are now wrapped in domain-specific decode exceptionsMalformed fallback addresses are skipped rather than aborting or crashing
a57ef6e7by Felix+707−9610 files
No security note in commit
Low 40 AI analysisMessage 36 · Opaque
EL ElectrumElectrum BitcoinSoftware wallets

bolt11: followup #10940

This commit tightens how Electrum parses BOLT11 Lightning invoices. It now rejects duplicate 'n' (node pubkey) tags instead of silently keeping only the first one, and it rejects invoice timestamps that are negative or too far in the futur…

Stricter validation of invoice timestamp boundsDuplicate 'n' tag now raises an exception instead of being silently droppedFollow-up to prior PR #10940, indicating a recent area of security-sensitive review
3026fbdbby f321x+14−232 files
No security note in commit
Low 32 AI analysisMessage 73 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10737 from sashazykov/hw-close-wallet-thread-leak

This commit fixes a shutdown bug in Electrum's hardware wallet support. Previously, if disconnecting from a hardware wallet failed, the cleanup thread could keep running, which could crash the application when closing. The fix stops the th…

Process abort/crash at shutdown due to leaked QThreadException during cleanup not handled, potentially causing abnormal terminationHardware wallet client close made best-effort with error logging
5b77ca46by Felix+9−22 files
No security note in commit
Informational 23 AI analysisMessage 58 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10958 from SomberNight/202609_deps

This commit is a routine dependency update for the Electrum Bitcoin wallet. It bumps versions of Python, OpenSSL, secp256k1, and various Python packages used in builds and runtime. The commit itself does not fix any known security bug in E…

Dependency version bumps for cryptographic libraries (OpenSSL, libsecp256k1, electrum-ecc)Routine maintenance commit with no explicit security claimHash-only changes in deterministic requirements files
8bd8c3fbby ghost43+57−5612 files
No security note in commit
Informational 21 AI analysisMessage 58 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10965 from SomberNight/202609_header_checkpoints

This commit only adds new Bitcoin block header checkpoints to Electrum's built-in chain data files. Checkpoints are hardcoded reference points that help the wallet verify it is following the real blockchain and not a fake one. Updating the…

Hardcoded blockchain checkpoints extended for all supported networksNo executable code or cryptographic logic changedNo bug fix, privilege change, or input validation change present
9e0b7e64by ghost43+276−04 files
No security note in commit
Informational 15 AI analysisMessage 58 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

update block header checkpoints

This commit simply refreshes Electrum's built-in list of Bitcoin blockchain checkpoints. Checkpoints are known-good block hashes that help the wallet verify it is following the real chain. The update adds newer checkpoints for mainnet, sig…

e11c579bby SomberNight+276−04 files
No security note in commit
Low 46 AI analysisMessage 58 · Thin
EL ElectrumElectrum BitcoinSoftware wallets

Merge pull request #10945 from accumulator/stale_utxo_data_fixes

This commit fixes cases where Electrum's wallet cache could hold onto outdated information about coins and transactions after blockchain reorganizations or mempool changes. For example, if a transaction was previously thought to be mined b…

stale cached state after reorg/mempool evictionincorrect confirmation metadata displayed to userscache invalidation added at multiple state transitions
e20b24c0by Felix+132−239 files
No security note in commit
Informational 15 AI analysisMessage 60 · Adequate
EL ElectrumElectrum BitcoinSoftware wallets

transaction: add set_mined_info setter to TxInput

This commit is a simple code cleanup: it replaces two repeated lines that set a transaction input's block height and position with a single helper method called set_mined_info. There is no change in behavior, no bug fix, and no security re…

61f71acbby f321x+7−73 files
No security note in commit
Repository ledger

Explore captured commits

Expand any commit for its author, full message, clarity score, changed files, triage signals, analysis, and source link.

Lower-priorityqml: add top padding to nostr relay url listby f321x · 24d93420 · Apr 14, 2026 · 2 filesMessage 68 · AdequateInformational 15Details
Commit message · f321x

qml: add top padding to nostr relay url list

The first relay url was close to the top of the ElTextArea and looked a
bit sliced. Adding some padding makes it look better.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This commit is a purely cosmetic user-interface tweak. It adds extra spacing at the top of a text box that displays Nostr relay web addresses, so the first line no longer looks cut off. There is no security relevance.

Lower-prioritywindows: delete lightning URI hooks on uninstallby f321x · 78135ac8 · Apr 13, 2026 · 1 fileMessage 68 · AdequateLow 28Details
Commit message · f321x

windows: delete lightning URI hooks on uninstall

On install we register the URIs for bitcoin, lightning, lnurlp and lnurlw
in the registry but on uninstall we would only remove bitcoin again.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Low 28/100

This commit fixes the Windows uninstaller so it removes leftover registry entries for Lightning payment links (lightning, lnurlp, lnurlw), not just Bitcoin links. Before the fix, uninstalling Electrum left these entries behind, which could make Windows still think Electrum was the program to handle those links even after the software was gone.

Lower-prioritypi: don't match literal | char in regexesby f321x · a5f1a299 · Apr 13, 2026 · 1 fileMessage 60 · AdequateLow 29Details
Commit message · f321x

pi: don't match literal | char in regexes

The RE_EMAIL and RE_DOMAIN regexes matched the '|' character
which is probably not what they should do.

60/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Low 29/100

This commit fixes two email/domain recognition patterns so they no longer treat the vertical bar character '|' as a valid letter. Before the fix, a string like 'example.c|m' could be accepted as a domain or email ending, which could let malformed payment identifiers slip through validation. The change is a small correctness fix in input parsing, not a dramatic security hole on its own.

Lower-prioritypi: handle lud-17 URI payment identifierby f321x · ef702d74 · Apr 13, 2026 · 8 filesMessage 76 · AdequateInformational 23Details
Commit message · f321x

pi: handle lud-17 URI payment identifier

LNURL-W/P can also be encoded in lud-17 form instead of bech32.
https://github.com/lnurl/luds/blob/luds/17.md
e.g.
lnurlw://example.com/api/test123
lnurlp://example.com/api/test123

76/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
AI analysis · Informational 23/100

This commit adds support for a new style of LNURL web link (so-called LUD-17 format, e.g. lnurlp://example.com or lnurlw://example.com) in Electrum. It also registers Electrum as the handler for these new link types on Android, Windows, macOS, and Linux. The change is a feature addition, not a fix for a known vulnerability. There is no evidence in the commit or supplied references that this is a security patch or that it addresses any reported security issue.

Lower-prioritybip21: add comment listing URI scheme handler registrationsby SomberNight · 016c8b5f · Apr 11, 2026 · 1 fileMessage 73 · AdequateInformational 15Details
Commit message · SomberNight

bip21: add comment listing URI scheme handler registrations

It is sufficiently rare that we have to touch this stuff that I always have to re-discover where/how it is done. And it is impractical to grep for "bitcoin:" or "lightning:".

Putting this "master list" comment very close to the BITCOIN_BIP21_URI_SCHEME variable seems like a good spot - at least this is where I would look for it first.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This commit only adds a plain-code comment listing where Electrum registers 'bitcoin:' and 'lightning:' URI scheme handlers across different operating systems. No code behavior changes, no security fix or vulnerability introduced.

AI review queuedqml: allow renaming walletsby f321x · 8a12874c · Apr 8, 2026 · 3 filesMessage 66 · AdequateLow 30Details
Commit message · f321x

qml: allow renaming wallets

Allows to rename a wallet file from the QML Wallet Details view.
This seems like a feature we should support as the use-case of a wallet can
change or maybe the user didn't think about a proper name when setting
up the wallet. Especially with lightning channels it is not possible to
restore from seed to change the name.

Fixes #4377

66/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context✓ Links an issue, advisory, or supporting reference
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Low 30/100

This commit adds a feature that lets users rename their wallet file from the mobile-style QML user interface. It is a normal feature addition, not a security fix. The code does include some safety checks, but there are small gaps that could theoretically allow a misnamed or clashing wallet file. There is no evidence this was released as a security patch or credited to a security researcher.

Lower-priorityexchange rate: fix coingecko apiby f321x · 9827734a · Apr 8, 2026 · 1 fileMessage 68 · AdequateInformational 21Details
Commit message · f321x

exchange rate: fix coingecko api

The CoinGecko API failed as the Honduran Lempira currency returned
null as value, rendering the API unusable:
```
11.19 | I | exchange_rate.CoinGecko | getting fx quotes for EUR
11.41 | E | exchange_rate.CoinGecko | failed fx quotes: InvalidOperation([<class 'decimal.ConversionSyntax'>])
Traceback (most recent call last):
File "/var/home/user/code/vibecoding_vm/electrum/electrum/exchange_rate.py", line 87, in update_safe
self._quotes = await self.get_rates(ccy)
^^^^^^^^^^^^^^^^^^^^^^^^^
File "/var/home/user/code/vibecoding_vm/electrum/electrum/exchange_rate.py", line 449, in get_rates
return dict([(ccy.upper(), to_decimal(d['value']))
~~~~~~~~~~^^^^^^^^^^^^
File "/var/home/user/code/vibecoding_vm/electrum/electrum/util.py", line 243, in to_decimal
return Decimal(str(x))
decimal.InvalidOperation: [<class 'decimal.ConversionSyntax'>]
```

```
"hnl":{"name":"Honduran Lempira","unit":"L","value":null,"type":"fiat"}
```

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 21/100

This is a bug-fix patch for Electrum's exchange-rate feature. CoinGecko began sending a 'null' price for the Honduran Lempira, which caused Electrum to crash while converting exchange rates and made the entire CoinGecko price feed unusable. The fix simply skips any currency whose value is missing. There is no security vulnerability here—just a reliability fix.

Lower-priorityplugin: nwc: handle 'null' params in requestby f321x · fd230cf9 · Apr 7, 2026 · 1 fileMessage 60 · AdequateLow 29Details
Commit message · f321x

plugin: nwc: handle 'null' params in request

Some clients send 'params: null' instead of 'params: {}' or no
params key at all.

60/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body
AI analysis · Low 29/100

This commit fixes a small bug in Electrum's NWC (Nostr Wallet Connect) plugin where a client sending 'params: null' in a request could cause the server to crash or behave unexpectedly. The fix treats null the same as missing or empty parameters, and improves the error message if something still goes wrong.

Security candidateverifier.py: fix CVE-2012-2459: reject left-sibling duplicatesby SomberNight · 3d390742 · Apr 3, 2026 · 2 filesMessage 93 · StrongModerate 62Details
Commit message · SomberNight

verifier.py: fix CVE-2012-2459: reject left-sibling duplicates

Due to how the txid-commitment merkle tree used in the block headers is constructed, we need an extra check to be able to validate the *position* of a txid in a block.

I think this is low severity for us.

See https://bitcointalk.org/?topic=102395 :

> The Merkle hash implementation that Bitcoin uses to calculate the Merkle
> root in a block header is flawed in that one can easily construct multiple
> lists of hashes that map to the same Merkle root.
> For example, merkle_hash([a, b, c]) and merkle_hash([a, b, c, c]) yield
> the same result. This is because, at every iteration, the Merkle hash
> function pads its intermediate list of hashes with the last hash if the
> list is of odd length, in order to make it of even length.
>
> And so, the Merkle root function can be effectively preimaged by
> changing the input so that one of the intermediate lists is of even
> length with the last two elements equal (where originally it was
> of odd length with a last element equal to the earlier mentioned two).
> As was later noted, this extends to any input length that is
> not a power of two:
> merkle_hash([a, b, c, d, e, f]) == merkle_hash([a, b, c, d, e, f, e, f]).
> Note that to maintain the same root hash, the only flexibility that
> exists is duplication of elements.

Ported from https://github.com/Electron-Cash/Electron-Cash/commit/165146362b4cb0ad74770b36aca1f9acb2800195

Co-authored-by: bitcoincashautist <80100588+A60AB5450353F40E@users.noreply.github.com>

93/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Links an issue, advisory, or supporting reference
Why it was queued
explicit security languagedefensive validation
AI analysis · Moderate 62/100

This commit fixes a known Bitcoin weakness (CVE-2012-2459) in Electrum's light-client verification. Because Bitcoin's Merkle tree duplicates the last hash when a level has an odd number of items, an attacker can craft a block proof that makes a transaction appear to be at a different position than it really is. The patch rejects proofs where a duplicated value appears on the left side of a hash pair, which only happens in forged proofs. The commit also adds tests for the fix and re-labels an existing related defense for CVE-2017-12842.

AI review queuedadd comments about xpub encryptionby ThomasV · 7a6a39d1 · Apr 2, 2026 · 3 filesMessage 45 · ThinInformational 15Details
Commit message · ThomasV

add comments about xpub encryption

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
signing or wallet pathsecond-pass: security-sensitive path
AI analysis · Informational 15/100

This commit only adds explanatory comments and a developer TODO note. It does not change any actual encryption logic, access controls, or behavior of the Electrum wallet. There is no security fix or vulnerability introduced by this change.

Lower-prioritycontrib: check for unsigned apk in release.shby f321x · 7adc833f · Apr 2, 2026 · 1 fileMessage 68 · AdequateInformational 18Details
Commit message · f321x

contrib: check for unsigned apk in release.sh

release.sh expects signed apks. if a non-releasemanager uses
release.sh to build it will build the apks unsigned and then
rename them to the same name as the signed apks. However
if the apks have already been built separately and are still named
*-unsigned.apk it will not detect them and instead try to build them
again. Instead it should just rename them to *-release.apk as if built
directly through release.sh.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 18/100

This is a small build-script fix for Electrum's release process. It changes how the release script handles Android APK files when a non-release-manager runs it. Previously, the script would always rebuild an unsigned APK even if one already existed, which could waste time or cause confusion. Now it checks whether an unsigned APK already exists and, if so, skips the rebuild and just renames it. There is no direct security vulnerability in the changed code itself.

Lower-priorityupdate release notes for version 4.7.2by SomberNight · e71616e6 · Apr 1, 2026 · 2 filesMessage 45 · ThinInformational 15Details
Commit message · SomberNight

update release notes for version 4.7.2

45/100 · ThinMessage clarity
✓ Descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100

This commit only updates the project's release notes and bumps the version number from 4.7.1 to 4.7.2. It does not change any executable code, so it cannot introduce or fix a security vulnerability by itself. The release notes do mention several prior fixes and changes that were made in other commits, but this particular change is purely documentation and versioning.

Lower-priorityonion_message: let caller specify considered channels for blinded paths. This allows restricting blinded paths to channels that have sufficient receive capacity for payment.by Sander van Grieken · 3e3bffa4 · Apr 1, 2026 · 1 fileMessage 73 · AdequateLow 43Details
Commit message · Sander van Grieken

onion_message: let caller specify considered channels for blinded paths.
This allows restricting blinded paths to channels that have sufficient receive
capacity for payment.

NOTE: this might have privacy issues, as this can be used to probe channel capacity.
Maybe randomize leeway?

@f321x: changed to use scid alias in create_blinded_path

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Low 43/100

This commit changes how Electrum builds private Lightning Network routing hints called 'blinded paths.' It lets the wallet choose which channels to include in a blinded path, and when channels are explicitly chosen, it publishes a short channel ID (a public network identifier) instead of just the next node's public key. The commit message itself warns that this 'might have privacy issues, as this can be used to probe channel capacity.' In plain terms, a payment sender could learn more than intended about a receiver's channel balances, which is a known Lightning privacy concern. The change is not a full fix; it is a partial implementation that introduces a new probing risk while trying to solve a payment-reliability problem.

Lower-prioritysegwit_addr: bech32 decode without checksum optionby Sander van Grieken · 65fb7395 · Apr 1, 2026 · 2 filesMessage 50 · ThinInformational 18Details
Commit message · Sander van Grieken

segwit_addr: bech32 decode without checksum option

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 18/100

This commit adds an optional way to decode Bech32 strings without verifying their checksum. It is a small, controlled API change in Electrum's address-decoding helper, not a fix for a known vulnerability. The new option is off by default and only used in tests, so it does not appear to weaken normal wallet behavior on its own.

Lower-prioritymove blinding_privkey from onion_message to lnonionby Sander van Grieken · 9bcbbdd3 · Apr 1, 2026 · 3 filesMessage 50 · ThinInformational 15Details
Commit message · Sander van Grieken

move blinding_privkey from onion_message to lnonion

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 15/100

This commit simply moves an existing helper function called blinding_privkey from one file (onion_message.py) to another (lnonion.py) and updates the import statements and tests accordingly. There is no change to what the function does, no bug fix, and no security patch.

Lower-priorityonion_message: fix route construction to ipby f321x · 4254c9a0 · Apr 1, 2026 · 3 filesMessage 68 · AdequateLow 42Details
Commit message · f321x

onion_message: fix route construction to ip

Don't include first hop of the path,
this is the hop from us to the first node and we don't
need a payload for ourselves.

Also adds unittest checking this.

68/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Low 42/100

This commit fixes how Electrum builds encrypted routes for sending private messages over the Lightning network. Previously, the code accidentally included the sender's own first hop in the encrypted onion path, which could cause message delivery to fail or produce an invalid route. The fix removes that unnecessary first hop and adds a unit test to confirm the route is built correctly.

Lower-priorityonion_message: factor out get_blinded_paths_to_me from get_blinded_reply_paths. the former also calculates payinfo information for payment scenarios. include payment_relay struct for payment blinded_paths.by Sander van Grieken · b7a51284 · Apr 1, 2026 · 1 fileMessage 73 · AdequateInformational 11Details
Commit message · Sander van Grieken

onion_message: factor out get_blinded_paths_to_me from get_blinded_reply_paths.
the former also calculates payinfo information for payment scenarios.
include payment_relay struct for payment blinded_paths.

73/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
AI analysis · Informational 11/100

This commit refactors how Electrum builds private (blinded) communication and payment paths through the Lightning network. It splits a reply-path helper into a more general helper that can also build payment paths, and adds the routing/fee details required by the Lightning protocol for payment blinded paths. There is no direct evidence in the commit that this fixes a security bug; it appears to be feature/protocol-completion work.

Lower-priorityonion_message: split send_onion_message_toby f321x · 4134dc7b · Apr 1, 2026 · 1 fileMessage 58 · ThinInformational 15Details
Commit message · f321x

onion_message: split send_onion_message_to

Factor out code from `send_onion_message_to` into a separate
function `_create_route_to_introduction_point` to make it
easier to reason about it and more testable.

58/100 · ThinMessage clarity
✓ Descriptive subject✓ Provides detailed explanatory context
AI analysis · Informational 15/100

This commit is a simple code cleanup: it moves a block of route-building logic into a new helper function without changing what the program actually does. There is no security fix or behavior change visible in the diff.

Lower-prioritytest_onion_message: test get_blinded_paths_to_meby f321x · 8d4affa2 · Apr 1, 2026 · 1 fileMessage 60 · AdequateInformational 15Details
Commit message · f321x

test_onion_message: test get_blinded_paths_to_me

Add unittest to test the payment path of get_blinded_paths_to_me

60/100 · AdequateMessage clarity
✓ Descriptive subject✓ Provides an explanatory body✓ Mentions testing or verification
AI analysis · Informational 15/100

This commit only adds a new unit test for an existing Lightning Network feature. It does not change any production code, fix a bug, or alter security behavior. There is no security issue here.

Lower-priorityonion_message: verify LNPeerAddr returned as hint in NoRouteFoundby Sander van Grieken · 5c4fc2d7 · Apr 1, 2026 · 1 fileMessage 60 · AdequateInformational 12Details
Commit message · Sander van Grieken

onion_message: verify LNPeerAddr returned as hint in NoRouteFound

60/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Mentions testing or verification! No meaningful explanatory body
AI analysis · Informational 12/100

This commit only adds and updates a test file. It checks that when Electrum cannot find a Lightning route to a peer for an onion message, the error includes the peer's network address. There is no production code change and no security fix or vulnerability evident in the diff.

Lower-prioritytests: test_onion_message: mock LNWallet._add_peerby f321x · 2b6ad681 · Apr 1, 2026 · 1 fileMessage 75 · AdequateInformational 13Details
Commit message · f321x

tests: test_onion_message: mock LNWallet._add_peer

Mock LNWallet._add_peer so direct connection fallbacks don't
cause an exception.

75/100 · AdequateMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides an explanatory body✓ Mentions testing or verification
AI analysis · Informational 13/100

This commit only changes a test file. It adds a mock (a fake stand-in) for a Lightning Network wallet function so that a test of onion message handling doesn't crash when the code tries to fall back to a direct peer connection. There is no change to production code and no security fix or vulnerability is indicated.

Lower-priorityonion_message: iterate blinded paths for onion message requestsby Sander van Grieken · 2e0f2632 · Apr 1, 2026 · 2 filesMessage 50 · ThinInformational 21Details
Commit message · Sander van Grieken

onion_message: iterate blinded paths for onion message requests

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
AI analysis · Informational 21/100

This commit finishes a previously unfinished feature in Electrum's Lightning onion-message handling. Before, when sending an onion message, the code only tried a single destination or blinded path. Now it can accept a list of paths and try them one by one (round-robin) if a request needs to be retried. This is a robustness improvement, not a fix for an active security bug. There is no evidence in the commit or supplied references that this addresses a disclosed vulnerability.

Lower-priorityset restrictive unix umask application-wide by defaultby SomberNight · 7755d97a · Mar 27, 2026 · 1 fileMessage 93 · StrongModerate 52Details
Commit message · SomberNight

set restrictive unix umask application-wide by default

- I noticed we were creating the RPC server unix domain socket with 0o775.
Instead of hunting down each individual line we create files/dirs, we should
just set a restrictive umask by default. We can still use chmod to relax this
for individual files. -- but we should try to be secure by default
- note: bitcoin core does the same
https://github.com/bitcoin/bitcoin/blame/2fe76ed8324af44c985b96455a05c3e8bec0a03e/src/common/system.cpp#L92
- the umask is set in run_electrum as opposed to __init__.py so that we don't side-effect use-cases where electrum is imported as a library

93/100 · StrongMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context✓ Explains rationale or failure mode✓ Links an issue, advisory, or supporting reference
AI analysis · Moderate 52/100

This change makes Electrum create files and folders with stricter default permissions, so other users on the same computer cannot read or modify them. The developer noticed the RPC socket was being created with overly permissive permissions (0o775, meaning group members had full access), and decided to tighten the default for all files rather than fixing each case individually. It is a proactive hardening patch, not a fix for a reported exploit.

Security candidatedaemon: set restrictive permission on RPC-server unix domain socketby SomberNight · 9d204abf · Mar 27, 2026 · 1 fileMessage 50 · ThinModerate 65Details
Commit message · SomberNight

daemon: set restrictive permission on RPC-server unix domain socket

0600 instead of 0775.

50/100 · ThinMessage clarity
✓ Specific, descriptive subject✓ Names a concrete action or component! No meaningful explanatory body
Why it was queued
credential or privilege state
AI analysis · Moderate 65/100

Electrum's background daemon runs a local control server using a Unix domain socket. Previously, that socket file was created with permissions 0775, meaning any user in the same group could connect to it and potentially control the wallet. This commit changes the permissions to 0600, so only the wallet's owner can access the socket. The commit itself notes a possible race condition: for a brief moment after the socket is created and before the permissions are tightened, other users might still be able to connect.

Lower-priorityci: llm sec review: tweak trigger typesby SomberNight · 85ea6af5 · Mar 27, 2026 · 1 fileMessage 72 · AdequateInformational 15Details
Commit message · SomberNight

ci: llm sec review: tweak trigger types

looks like if there is an overlap between automatic and manual, manual wins

72/100 · AdequateMessage clarity
✓ Descriptive subject✓ Names a concrete action or component✓ Uses a recognizable type or scope✓ Provides an explanatory body
AI analysis · Informational 15/100

This commit adjusts a CI (continuous integration) configuration so that an automated LLM security-review task is triggered automatically only for users with write or admin permissions, while a manual trigger is used for everyone else. It is a workflow tweak with no direct effect on Electrum's wallet code, cryptography, or user-facing security.