Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mint: When melt fails there is an sql error attempting to add proofs #391

Open
thesimplekid opened this issue Oct 6, 2024 · 1 comment
Assignees
Labels
bug Something isn't working mint
Milestone

Comments

@thesimplekid
Copy link
Collaborator


9639e6
2024-10-06T20:13:01.009989Z DEBUG melt:verify_melt_request: cdk::mint::melt: Verified melt quote: b27fe966-5afa-4967-9073-ebb50b869730
2024-10-06T20:13:01.147059Z ERROR melt: cdk_cln: Error attempting to pay invoice: 1e81fe1582cea3def64f6a8ea0dc36c65f2c620c38b478193ee2279e309639e6
2024-10-06T20:13:01.147194Z ERROR melt: cdk_cln: Err(RpcError { code: Some(210), message: "Ran out of routes to try after 1 attempt: see `paystatus`", data: None })
2024-10-06T20:13:01.147263Z ERROR melt: cdk::mint::melt: Error returned attempting to pay: b27fe966-5afa-4967-9073-ebb50b869730 Wrong CLN response
2024-10-06T20:13:01.148733Z  INFO melt: cdk::mint::melt: Lightning payment for quote b27fe966-5afa-4967-9073-ebb50b869730 failed.
2024-10-06T20:13:01.424414Z DEBUG process_swap_request: cdk_sqlite::mint: Attempting to add known proof. Skipping.... SQLX(Database(SqliteError { code: 1555, message: "UNIQUE constraint failed: proof.y" }))

@thesimplekid thesimplekid added bug Something isn't working mint labels Oct 6, 2024
@thesimplekid thesimplekid self-assigned this Oct 7, 2024
@thesimplekid thesimplekid changed the title Mint: When melt failts there is an sql error attempting to add proofs Mint: When melt fails there is an sql error attempting to add proofs Oct 12, 2024
@thesimplekid thesimplekid added this to the v0.5.0 milestone Oct 12, 2024
@crodas
Copy link
Contributor

crodas commented Oct 14, 2024

Is the same proof being added twice? Either ON CONFLICT or UPSERT should be used instead.

I think that ON CONFLICT IGNORE, since errors are not fatal, but take my point with a grain of salt. I still have very little context for the project, but I'm learning more daily.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working mint
Projects
None yet
Development

No branches or pull requests

2 participants