cfallin commented on issue #3616:
Urgh, I forgot the one change in
Cargo.toml
that actually makessha2
optional; fix coming momentarily!
alexcrichton commented on issue #3616:
If
sha2
is uncomfortable enough to have a PR to implement this then it seems like we would want to disable this feature in the Wasmtime embedding, but that would then basically subvert the purpose of adding this check in the first place. If we take this even further than the feature here should be on-by-default for development but off-by-default for crates.io publications in the limit.Overall I feel like we either need to find a slimmer crate so that it doesn't matter or we need to consider these concerns for Wasmtime as well. Personally I'd be fine just using
std::hash::SipHasher
since it's at least built-in to std and despite being deprecated it won't ever be removed.
cfallin commented on issue #3616:
That's a good point! I took the
SipHash
approach instead in #3619; closing this PR.
Last updated: Dec 23 2024 at 12:05 UTC