Skip to content

Avoid date string allocation in OpenSslX509CertificateReader.ExtractValidityDateTime #75138

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 10, 2022
Merged

Avoid date string allocation in OpenSslX509CertificateReader.ExtractValidityDateTime #75138

merged 2 commits into from
Sep 10, 2022

Conversation

stephentoub
Copy link
Member

Also consolidated some duplicated code.

@stephentoub
Avoid date string allocation in OpenSslX509CertificateReader.ExtractV…
5be3e96 
…alidityDateTime

Also consolidated some duplicated code.
@ghost ghost added the area-System.Security label Sep 6, 2022
@ghost ghost assigned stephentoub Sep 6, 2022
@ghost
Copy link

ghost commented Sep 6, 2022

Tagging subscribers to this area: @dotnet/area-system-security, @vcsjones
See info in area-owners.md if you want to be subscribed.

Issue Details

Also consolidated some duplicated code.

Author: stephentoub
Assignees: -
Labels:

area-System.Security
Milestone: -

@stephentoub stephentoub merged commit aaf0bde into dotnet:main Sep 10, 2022
@stephentoub stephentoub deleted the opensslasciidatestring branch September 10, 2022 16:18
@ghost ghost locked as resolved and limited conversation to collaborators Oct 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bartonjs bartonjs bartonjs approved these changes

Assignees

@stephentoub stephentoub

Labels
area-System.Security
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@stephentoub @bartonjs