Skip to content

Conversation

@eyusufatik
Copy link
Contributor

The mentioned commit in the changelog is this: c1bb316

Change looks unnecessary, however, I had a minor heart-attack once I saw the changelog because we've been using the global-context feature for some time.

@apoelstra
Copy link
Member

Yep, good catch!

And for what it's worth, I hate this name global-context-less-secure. The "less secure" global context is only "less secure" assuming a cascade of other failures which we have no evidence have ever happened, and assuming a physical attacker who can produce hundreds or thousands of signatures. So I apologize for your moment of panic.

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

ACK 4b00c71; successfully ran local tests

@apoelstra apoelstra merged commit 073e2af into rust-bitcoin:master Jan 13, 2025
29 of 30 checks passed
chain-forgexcr45 added a commit to chain-forgexcr45/rust-secp256k1 that referenced this pull request Sep 28, 2025
4b00c71c157c61d50c409e2b67e9e7935b03679a global-context-less-secure enables global-context, not the other way around (eyusufatik)

Pull request description:

  The mentioned commit in the changelog is this: rust-bitcoin/rust-secp256k1@c1bb316

  Change looks unnecessary, however, I had a minor heart-attack once I saw the changelog because we've been using the global-context feature for some time.

ACKs for top commit:
  apoelstra:
    ACK 4b00c71c157c61d50c409e2b67e9e7935b03679a; successfully ran local tests

Tree-SHA512: 68af4a4257abee5a92157d638fa4c7ac4855aba98b45c558adf7ae22edd478f9d50b39d2ab6f708f6b1963b84fd25127930b0e7dccc6eb0c32302b684b41e678
william2332-limf added a commit to william2332-limf/rust-secp256k1 that referenced this pull request Oct 2, 2025
4b00c71c157c61d50c409e2b67e9e7935b03679a global-context-less-secure enables global-context, not the other way around (eyusufatik)

Pull request description:

  The mentioned commit in the changelog is this: rust-bitcoin/rust-secp256k1@c1bb316

  Change looks unnecessary, however, I had a minor heart-attack once I saw the changelog because we've been using the global-context feature for some time.

ACKs for top commit:
  apoelstra:
    ACK 4b00c71c157c61d50c409e2b67e9e7935b03679a; successfully ran local tests

Tree-SHA512: 68af4a4257abee5a92157d638fa4c7ac4855aba98b45c558adf7ae22edd478f9d50b39d2ab6f708f6b1963b84fd25127930b0e7dccc6eb0c32302b684b41e678
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