Skip to content

x509-cert: wip: owned api #776

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

Closed
wants to merge 38 commits into from
Closed

Conversation

baloo
Copy link
Member

@baloo baloo commented Nov 28, 2022

This is a wip on top of #769 to make x509-cert an owned api.

I plan on extracting commits from here in independent, smaller PRs.

@baloo baloo force-pushed the baloo/owned-api/v2 branch 2 times, most recently from a71364a to 90eb5e8 Compare November 29, 2022 04:49
tarcieri and others added 28 commits November 30, 2022 12:30
NOTE: breaking change.

Previously `AlgorithmIdentifier::parameters` were always `AnyRef`.
This commit changes them to a generic parameter `Params`.

An alias `AlgorithmIdentifierRef` provides a type identical to the
original with `AnyRef` as its `parameters`, which is used in all of the
other crates in this repo.
@baloo baloo force-pushed the baloo/owned-api/v2 branch from 90eb5e8 to d584ff0 Compare November 30, 2022 20:38
@baloo
Copy link
Member Author

baloo commented Dec 2, 2022

@tarcieri this last commit makes the x509-cert vary on its parameters for the storage. I also did some work on the derive crate, but I'm not too happy with the result. If you have suggestions ...

@baloo baloo closed this Dec 18, 2022
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