Comments (5)
The additional semantic requirements for Eq
are laid out in the trait's docs. I think this is okay to implement at least for any of our types that are just "bags of bytes". The three relevant properties are:
- reflexivity:
a == a
- symmetry:
a == b
impliesb == a
- transitivity:
a == b
andb == c
impliesa == c
The type system will enforce that the bag-of-bytes types can only be compared with the same type, which guarantees that the underlying bytes semantically mean the same thing and that we're not, for example, comparing a private key and a hash. That's the only thing I can think of that might break one of the above properties, and it's not an issue.
from orion.
Just for reference, clippy suggests the additional Eq
on the following types:
- X25519
PublicKey
+PrivateKey
+SessionKeys
(high-level API) - BLAKE2b
Hasher
enum
Good points @vlmutolo. Seems like this shouldn't be too much of an issue then.
from orion.
In the const generics refactor, the byte containers will all look like Public<C, D>
. I think we can derive Eq
on PublicData
and the various D
storage types (e.g. ArrayData
). This means that Public
will implement Eq
if and only if the context parameter C
implements Eq
. So we can control which types implement Eq
just using the context parameter, which is in line with the spirit of the refactor: pushing logic (such as min, max sizes) into the context parameters and letting everything else be the same.
I imagine we'll want to implement Eq
for all the current byte container types, but it's good that we'll have the option to not implement it selectively.
from orion.
Do you think it'd be better to postpone this to post const-generics, that you've been working on @vlmutolo?
from orion.
I think we can go ahead and add this in the current implementation. I'll also make sure to add it in the const-generics branch.
from orion.
Related Issues (20)
- `nightly` doctest builds fail HOT 3
- Missing `Debug` trait for BLAKE2b `Hasher` enum HOT 3
- MSRV `1.52.0` fails because of bad version of `plotters`
- Investigate whether actions-rs should be replaced HOT 1
- Remove ignores of `criterion`-based RUSTSEC advisories once fixed HOT 2
- Misuse issue in (X)ChaCha20 and (X)ChaCha20-Poly1305 APIs HOT 2
- Remove `cargo audit` workflow HOT 1
- Making zeroization optional
- Adding key-committing AEAD implementations (CTX or UtC/RtC/HtE) HOT 7
- Consider using `cargo-semver-checks`
- Provide link to OWASP in Argon2i documentation
- Update Wycheproof test vectors
- Add chunked input processing of KATs
- Unify hashing APIs of BLAKE2b, SHA2 and SHA3
- Asymmetric encryption functions for long-term / re-usable encrypted data storage HOT 5
- Add support for HPKE (ChaCha20-Poly1305 + DHKEM(X25519, HKDF-SHA256))
- `Digest` to `&[u8]` HOT 2
- Does orion support iOS and Android platforms? HOT 1
- Consider using checked_mul(8) instead of checked_shl(3) in SHA2 HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from orion.