Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

elliptic-curve: use sec1 crate's EncodedPoint #771

Merged
merged 1 commit into from
Sep 19, 2021

Conversation

tarcieri
Copy link
Member

The EncodedPoint type was extracted to the sec1 crate in:

RustCrypto/formats#45

This switches the elliptic-curve to using the newly extracted implementation.

This was a bit tricky/invasive as now EncodedPoint now only knows a size and not the rest of the details of the curve.

APIs that were previously heavily leveraging EncodedPoint should probably switch to the PublicKey type instead unless they are specifically intending to work with SEC1-encoded points.

@tarcieri tarcieri force-pushed the elliptic-curve/use-sec1-crate-encoded-point branch from 4b9b843 to 62bdd01 Compare September 19, 2021 18:24
The `EncodedPoint` type was extracted to the `sec1` crate in:

RustCrypto/formats#45

This switches the `elliptic-curve` to using the newly extracted
implementation.

This was a bit tricky/invasive as now `EncodedPoint` now only knows a
size and not the rest of the details of the curve.

APIs that were previously heavily leveraging `EncodedPoint` should
probably switch to the `PublicKey` type instead unless they are
specifically intending to work with SEC1-encoded points.
@tarcieri tarcieri force-pushed the elliptic-curve/use-sec1-crate-encoded-point branch from 62bdd01 to db13593 Compare September 19, 2021 18:36
@tarcieri tarcieri merged commit 5251de6 into master Sep 19, 2021
@tarcieri tarcieri deleted the elliptic-curve/use-sec1-crate-encoded-point branch September 19, 2021 18:52
tarcieri added a commit to RustCrypto/signatures that referenced this pull request Sep 19, 2021
tarcieri added a commit to RustCrypto/signatures that referenced this pull request Sep 19, 2021
Incorporates upstream changes from:

RustCrypto/traits#771
tarcieri added a commit that referenced this pull request Sep 20, 2021
This was removed in #771 but can still be supported
tarcieri added a commit that referenced this pull request Sep 20, 2021
)

This was removed in #771 but can still be supported
tarcieri added a commit to RustCrypto/elliptic-curves that referenced this pull request Sep 20, 2021
These code changes are necessary to support a migration to the `sec1`
crate, which extracted types like `EncodedPoint` into a separate
reusable crate.

Relevant `elliptic-curve` PR:

RustCrypto/traits#771
tarcieri added a commit to RustCrypto/elliptic-curves that referenced this pull request Sep 20, 2021
These code changes are necessary to support a migration to the `sec1`
crate, which extracted types like `EncodedPoint` into a separate
reusable crate.

Relevant `elliptic-curve` PR:

RustCrypto/traits#771
@tarcieri tarcieri mentioned this pull request Nov 20, 2021
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.

1 participant