Skip to content

Conversation

@tarcieri
Copy link
Member

Consolidates the following former three traits into a single trait:

  • AffineArithmetic
  • ProjectiveArithmetic
  • ScalarArithmetic

It doesn't make sense to impl one of these traits without impl'ing them all, so this commit combines them into a single trait.

@tarcieri tarcieri force-pushed the elliptic-curve/curve-arithmetic-trait branch from 8131fa1 to 810478b Compare January 14, 2023 03:18
Consolidates the following former three traits into a single trait:

- `AffineArithmetic`
- `ProjectiveArithmetic`
- `ScalarArithmetic`

It doesn't make sense to impl one of these traits without impl'ing them
all, so this commit combines them into a single trait.
@tarcieri tarcieri force-pushed the elliptic-curve/curve-arithmetic-trait branch from 810478b to 921a4f9 Compare January 14, 2023 03:20
@tarcieri tarcieri merged commit df3f85b into master Jan 14, 2023
@tarcieri tarcieri deleted the elliptic-curve/curve-arithmetic-trait branch January 14, 2023 03:24
@tarcieri tarcieri mentioned this pull request Mar 1, 2023
sorah added a commit to sorah/needroleshere that referenced this pull request Nov 4, 2023
Resolve incompatibilities as follows:

- src/certificate.rs: RustCrypto/formats#771
- src/cmd/serve.rs: Move to FromRequestParts and use headers crate

- src/ecdsa_sha256.rs: RustCrypto/traits#1196
  and RustCrypto/signatures#574

- src/error.rs: Remove Duplicates

- src/identity.rs: Remove owned validation logic as it should be done at
    x509_cert::serial_number::SerialNumber<x509_cert::certificate::Rfc5280>

- src/sign.rs: RustCrypto/RSA#244
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