Skip to content

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 <jon_p_griffiths@yahoo.com>
Changelog-None

Signed-off-by: Jon Griffiths <jon_p_griffiths@yahoo.com>
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 <jon_p_griffiths@yahoo.com>
@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