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

Wally 1.4.0 update #8158

Merged
merged 3 commits into from
Mar 20, 2025
Merged

Conversation

jgriffiths
Copy link
Contributor

@jgriffiths jgriffiths commented Mar 12, 2025

Not an important update, but is required for Elements p2tr support whenever that is due.

Supersedes #8098 using the just minted wally release. Removes now-redundant serialized psbt copies, and enables signature hash caching for faster PSBT signing.

The code that creates signature hashes with transactions should be migrated to PSBT, it's kind of a mix currently and the APIs exposed/wally APIs used don't lend themselves to caching or taproot signing. Thats out of scope for this PR though.

Important

25.02 FREEZE JANUARY 31ST: Non-bugfix PRs not ready by this date will wait for 25.05.

Checklist

Before submitting the PR, ensure the following tasks are completed. If an item is not applicable to your PR, please mark it as checked:

  • The changelog has been updated in the relevant commit(s) according to the guidelines.
  • Tests have been added or modified to reflect the changes.
  • Documentation has been reviewed and updated as needed.
  • Related issues have been listed and linked, including any that this PR closes.

Changelog-Changed: Update libwally to 1.4.0

Signed-off-by: Jon Griffiths <[email protected]>
Enabling the cache makes signing significantly faster for segwit inputs,
particularly taproot which was designed with caching in mind.

Changelog-None

Signed-off-by: Jon Griffiths <[email protected]>
@jgriffiths jgriffiths changed the title Wally 1 4 0 update Wally 1.4.0 update Mar 12, 2025
Copy link
Contributor

@rustyrussell rustyrussell left a comment

Choose a reason for hiding this comment

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

Ack beffd02

@rustyrussell rustyrussell merged commit 8fc4cb6 into ElementsProject:master Mar 20, 2025
40 checks passed
@jgriffiths jgriffiths deleted the wally_1_4_0_update branch March 21, 2025 00:52
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