AI-generated analysisPublished automatically and not human-verified. Validated context appears in community notes below.
← Watch feed
Informational 19 Bitcoin

bkpr: remove different currency support.

Public commit record

What the developer wrote

Authored by Rusty Russell

68/100 · Adequate
bkpr: remove different currency support.

We still output the fields, they're just always the currency of the node.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
Changelog-Changed: Plugins: `bookkeeper` now explicitly assumes every transaction is in the same currency as the node (true unless you added manually)
✓ Descriptive subject✓ Names a concrete action or component✓ Provides detailed explanatory context
The short version

What changed, and why it matters

This commit removes the bookkeeper plugin's ability to track multiple currencies. From now on, every transaction is reported in the node's own currency (for example, Bitcoin on mainnet). The change simplifies the code and database schema but does not fix a security bug. It could affect accounting accuracy for anyone who had manually configured different currencies.

Recommended action

No immediate security action required. Operators relying on bookkeeper multi-currency reporting should review exported data and migration notes, since historical currency fields are no longer used.

Security signals we found

01

No memory-safety, authentication, authorization, or cryptographic changes observed

02

Behavioral change: currency metadata is now always the node's chainparams->lightning_hrp instead of a per-event value

03

Database schema usage changes: currency columns are dropped from SELECT/INSERT statements

04

Potential data-fidelity concern for users who manually added non-node currency events

Risk score

Why this scored 19/100

Our methodology →
Potential impact 2/30
Exploitability 0/25
Stealth signal 0/15
Affected reach 5/15
Confidence 8/10
Evidence quality 4/5
Human-validated context

Community notes

Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.

No validated notes yet.

The AI analysis stands alone for now. Submit a note if you can add evidence or important context.