Skip to content

fix: melt from untrusted mint#211

Merged
a1denvalu3 merged 1 commit intocashubtc:mainfrom
thesimplekid:melt_from_untrusted
Mar 9, 2026
Merged

fix: melt from untrusted mint#211
a1denvalu3 merged 1 commit intocashubtc:mainfrom
thesimplekid:melt_from_untrusted

Conversation

@thesimplekid
Copy link
Copy Markdown
Contributor

This uses the melt with proofs and prevents a swap before the melt In the case of the unknown mint we create a temp wallet and do not receive those proofs to the wallet. Instead we melt directly with them.

We do not want a swap before the melt to prevent a partial operation. If we swap but the melt fails we have spent the senders tokens but we have not received them to our wallet as we do not trust there mint

This uses the melt with proofs and prevents a swap before the melt
In the case of the unknown mint we create a temp wallet and do not
receive those proofs to the wallet. Instead we melt directly with them.

We do not want a swap before the melt to prevent a partial operation.
If we swap but the melt fails we have spent the senders tokens
but we have not received them to our wallet as we do not trust there mint
@thesimplekid thesimplekid requested a review from a1denvalu3 March 9, 2026 09:52
@github-project-automation github-project-automation bot moved this to Backlog in Numo Mar 9, 2026
@a1denvalu3
Copy link
Copy Markdown
Collaborator

Tested ACK

@a1denvalu3 a1denvalu3 merged commit fa55cde into cashubtc:main Mar 9, 2026
2 checks passed
@github-project-automation github-project-automation bot moved this from Backlog to Done in Numo Mar 9, 2026
@thesimplekid thesimplekid deleted the melt_from_untrusted branch March 9, 2026 11:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants