Skip to content

x509-cert: make Certificate an owned type - #806

Merged
tarcieri merged 27 commits into
RustCrypto:masterfrom
baloo:baloo/owned-api/certificate
Dec 30, 2022
Merged

x509-cert: make Certificate an owned type#806
tarcieri merged 27 commits into
RustCrypto:masterfrom
baloo:baloo/owned-api/certificate

Conversation

@baloo

Copy link
Copy Markdown
Member

As discussed in #803 let's make Certificate an owned type. We'll revisit that in a follow up PR to accept a trait Profile as parameter allowing it to be either a reference or an owned type.

@baloo
balooforce-pushed the baloo/owned-api/certificate branch from ba1164d to c1c0d47CompareDecember 18, 2022 18:16
Comment threadder/src/string.rs Outdated
Comment threadder/src/asn1/ia5_string.rs Outdated
@baloo
balooforce-pushed the baloo/owned-api/certificate branch 5 times, most recently from a215178 to 1188fcfCompareDecember 19, 2022 03:51
@tarcieri

Copy link
Copy Markdown
Member

@baloo can you rebase?

@baloo
balooforce-pushed the baloo/owned-api/certificate branch from 1188fcf to f41a358CompareDecember 30, 2022 04:26
@baloo

Copy link
Copy Markdown
MemberAuthor

@tarcieri done

@baloo
balooforce-pushed the baloo/owned-api/certificate branch from 81c741d to 2109a1bCompareDecember 30, 2022 04:43
@tarcieri
tarcieri merged commit eab0dab into RustCrypto:masterDec 30, 2022
@tarcieri

Copy link
Copy Markdown
Member

Thank you!

npmccallum added a commit to npmccallum/formats that referenced this pull request Jan 9, 2023
RustCrypto#806
The above PR made most types in this crate owned. However, some types were
missed. For consistencey, complete the migration.
Signed-off-by: Nathaniel McCallum <nathaniel@profian.com>
npmccallum added a commit to npmccallum/formats that referenced this pull request Jan 9, 2023
RustCrypto#806
The above PR made most types in this crate owned. However, some types were
missed. For consistencey, complete the migration.
Signed-off-by: Nathaniel McCallum <nathaniel@profian.com>
npmccallum added a commit to npmccallum/formats that referenced this pull request Jan 9, 2023
RustCrypto#806
The above PR made most types in this crate owned. However, some types were
missed. For consistencey, complete the migration.
Signed-off-by: Nathaniel McCallum <nathaniel@profian.com>
npmccallum added a commit to npmccallum/formats that referenced this pull request Jan 9, 2023
RustCrypto#806
The above PR made most types in this crate owned. However, some types were
missed. For consistencey, complete the migration.
Signed-off-by: Nathaniel McCallum <nathaniel@profian.com>
npmccallum added a commit to npmccallum/formats that referenced this pull request Jan 9, 2023
RustCrypto#806
The above PR made most types in this crate owned. However, some types were
missed. For consistencey, complete the migration.
Signed-off-by: Nathaniel McCallum <nathaniel@profian.com>
tarcieri pushed a commit that referenced this pull request Jan 10, 2023
#806
The above PR made most types in this crate owned. However, some types were
missed. For consistency, complete the migration.
Removed alloc feature: this crate unconditionally requires alloc.
Signed-off-by: Nathaniel McCallum <nathaniel@profian.com>
@lumaglumag mentioned this pull request Apr 4, 2023
scv35 pushed a commit to scv35/format-encoders-decoders that referenced this pull request Jul 4, 2025
RustCrypto/formats#806
The above PR made most types in this crate owned. However, some types were
missed. For consistency, complete the migration.
Removed alloc feature: this crate unconditionally requires alloc.
Signed-off-by: Nathaniel McCallum <nathaniel@profian.com>
ghost-knightx56yui added a commit to ghost-knightx56yui/formats that referenced this pull request Nov 13, 2025
RustCrypto/formats#806
The above PR made most types in this crate owned. However, some types were
missed. For consistency, complete the migration.
Removed alloc feature: this crate unconditionally requires alloc.
Signed-off-by: Nathaniel McCallum <nathaniel@profian.com>
Sign up for freeto 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

@baloo@tarcieri