taproot-primitives: Enable crypto features
What the developer wrote
Authored by Mitchell Bagot
taproot-primitives: Enable crypto features
The crypto dep of taproot-primitives is currently used with no features.
While no features are currently required for the limited use of crypto
in the crate, this may change, so the features of crypto should be
enabled based on the features of taproot-primitives.
Add feature enabling on crypto for all taproot-primitives features.y
What changed, and why it matters
This is a small Cargo.toml configuration change that forwards feature flags from the taproot-primitives crate to its internal bitcoin-crypto dependency. It does not change any executable code, fix a bug, or alter behavior under current feature usage. The commit is framed as a future-proofing measure, not a security fix.
No security action required. Treat as routine build/configuration maintenance.
Security signals we found
No source-code changes
No memory-safety, cryptographic, or input-validation changes
No bug fix or vulnerability remediation described in commit message
Feature-gating only; no runtime behavior change
Evidence from the diff
The commit modifies taproot_primitives/Cargo.toml so that every taproot-primitives feature (std, alloc, serde, arbitrary, hex) also enables the corresponding feature on the bitcoin-crypto dependency. Previously bitcoin-crypto was used with no features enabled. This is a dependency feature-propagation cleanup; no Rust source code is changed and no existing API behavior is modified.
Changed components
taproot_primitives/Cargo.tomlInspect captured patch +5 / −5
diff --git a/taproot_primitives/Cargo.toml b/taproot_primitives/Cargo.toml
index 1b04c551..c5d42dce 100644
--- a/taproot_primitives/Cargo.toml
+++ b/taproot_primitives/Cargo.toml
@@ -14,11 +14,11 @@ exclude = ["tests", "contrib"]
[features]
default = ["std", "hex"]
-std = ["alloc", "hashes/std", "internals/std", "serde?/std", "secp256k1/std"]
-alloc = ["hashes/alloc", "internals/alloc", "secp256k1/alloc"]
-serde = ["dep:serde", "hashes/serde", "internals/serde", "secp256k1/serde"]
-arbitrary = ["dep:arbitrary", "secp256k1/arbitrary"]
-hex = ["hashes/hex"]
+std = ["alloc", "crypto/std", "hashes/std", "internals/std", "serde?/std", "secp256k1/std"]
+alloc = ["crypto/alloc", "hashes/alloc", "internals/alloc", "secp256k1/alloc"]
+serde = ["dep:serde", "crypto/serde", "hashes/serde", "internals/serde", "secp256k1/serde"]
+arbitrary = ["dep:arbitrary", "crypto/arbitrary", "secp256k1/arbitrary"]
+hex = ["crypto/hex", "hashes/hex"]
[dependencies]
crypto = { package = "bitcoin-crypto", path = "../crypto", version = "0.2.0", default-features = false }
Why this scored 12/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.