diff options
| author | Adam Soltys <[email protected]> | 2026-05-17 23:27:01 -0700 |
|---|---|---|
| committer | Adam Soltys <[email protected]> | 2026-05-17 23:27:01 -0700 |
| commit | 9fca61790af069dc150a6923c89d119c537e512c (patch) | |
| tree | 887cadb91d1956ab945d2e2568cd6998457dc813 /scripts/cln_schema.sql | |
| parent | b52eee57bd9055d0f2b9044d6b87ce90ca5e48cc (diff) | |
| download | coinos-server-9fca61790af069dc150a6923c89d119c537e512c.tar.gz coinos-server-9fca61790af069dc150a6923c89d119c537e512c.zip | |
sendLightning / sendKeysend: only reverse when CLN reports the payment failed
The catch blocks for ln.xpay and ln.keysend reversed the user's debit
whenever the RPC call threw, regardless of the actual payment state in
CLN. If the RPC errored transiently while the payment was still pending,
the user's balance got refunded — and then the lightning payment could
still complete, leaving the user with both the refund and the outgoing
sats.
Both call sites now mirror the safe check() logic:
const failed = !pays.length || pays.every((p) => p.status === "failed");
if (completed) finalize ...
else if (failed) reverse ...
// else (pending) leave the debit; check() reconciles when CLN settles.
Co-Authored-By: Claude Opus 4.7 (1M context) <[email protected]>
Diffstat (limited to 'scripts/cln_schema.sql')
0 files changed, 0 insertions, 0 deletions