Skip to content

Comments

Ln verification do not use payment_hash as verification identifier#10

Merged
86667 merged 3 commits intomasterfrom
ln_verification_unique_ids
Jan 5, 2026
Merged

Ln verification do not use payment_hash as verification identifier#10
86667 merged 3 commits intomasterfrom
ln_verification_unique_ids

Conversation

@86667
Copy link
Contributor

@86667 86667 commented Jan 2, 2026

Create a new Id uuid on ln verifications instead of using payment_hash.

Also adds ln_verificaiton/price endpoint to fetch configured invoice price.

Breaking change

This db schema change is breaking in Staging. (Production is not yet deployed)
I'll manually drop the tables before depploying:

DROP TABLE IF EXISTS lightning_verifications;
DELETE FROM migrations WHERE name = 'm20251216_create_ln_verification';

@86667 86667 force-pushed the ln_verification_unique_ids branch from 01f9477 to bfcc292 Compare January 5, 2026 09:41
Copy link
Collaborator

@SeverinAlexB SeverinAlexB left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's go. Code looks flawless 🚀

@86667 86667 merged commit 91a2175 into master Jan 5, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants