Skip to content

spki: borrow the public key in spki::from_key - #1290

Merged
baloo merged 1 commit into
RustCrypto:masterfrom
baloo:baloo/spki/spki-from_key
Jan 4, 2024
Merged

spki: borrow the public key in spki::from_key#1290
baloo merged 1 commit into
RustCrypto:masterfrom
baloo:baloo/spki/spki-from_key

Conversation

@baloo

@baloobaloo commented Jan 3, 2024

Copy link
Copy Markdown
Member

SubjectPublicKeyInfoOwned does not require the ownership of the key.

This is a breaking change.

@baloobaloo added the breaking Change to be merged with next release cycle label Jan 3, 2024
@baloo

baloo commented Jan 3, 2024

Copy link
Copy Markdown
MemberAuthor

Guess I'll need to break this in two.
Release a minor of spki with the first change, then bring that in x509-cert in a follow up.

@baloobaloo mentioned this pull request Jan 3, 2024
@baloo

baloo commented Jan 3, 2024

Copy link
Copy Markdown
MemberAuthor

I am surprised to see minimal version of x509 work here. I expected it to fail, is this not a breaking change?!

@tarcieri

Copy link
Copy Markdown
Member

Not sure how it would be possible?

@baloo
balooforce-pushed the baloo/spki/spki-from_key branch from 9df790c to 43f0929CompareJanuary 4, 2024 18:47
baloo added a commit to baloo/formats that referenced this pull request Jan 4, 2024
`SubjectPublicKeyInfoOwned` does not require the ownership of the key.
This is a breaking change.
@baloo
balooforce-pushed the baloo/spki/spki-from_key branch from 43f0929 to 24ef7f6CompareJanuary 4, 2024 19:47
@baloo
baloo merged commit a40e97b into RustCrypto:masterJan 4, 2024
@baloo
baloo deleted the baloo/spki/spki-from_key branch January 4, 2024 20:37
baloo added a commit to baloo/formats that referenced this pull request Jan 4, 2024
@tarcieritarcieri mentioned this pull request Apr 3, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breakingChange to be merged with next release cycle

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@baloo@tarcieri