Skip to content

Conversation

@jonasnick
Copy link
Collaborator

Fixes #40

CHANGELOG.md Outdated

# 0.5.0 - 2021-10-22

- IMPORTANT: Encrypt ECDSA adaptor signatures in release builds. Previously adaptor signatures were NOT encrypted.
Copy link
Contributor

Choose a reason for hiding this comment

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

This sounds like we were outputting non-encrypted adaptor signatures (which is not a thing) whereas really we were just returning 62 zero bytes.

I think the fix is important but this description is a bit misleading.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Good point. Pushed fix attempt.

Copy link
Contributor

@thomaseizinger thomaseizinger left a comment

Choose a reason for hiding this comment

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

ACK 582a5a0

@jonasnick jonasnick merged commit c5a074a into BlockstreamResearch:master Nov 4, 2021
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.

New release

2 participants