Skip to content

Dead end S.Security.Cryptography.OpenSsl package #51854

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

Merged
merged 3 commits into from
Apr 26, 2021

Conversation

ViktorHofer
Copy link
Member

@ViktorHofer ViktorHofer commented Apr 26, 2021

To reduce complexity of the cryptography OpenSsl library, dead ending
its package as most of the configurations are partial facades anyway and
adding OpenSsl to the targeting pack. It's already part of the runtime
and exposed in aspnetcore's targeting pack. Note that the Platform
Analyzer will discourage the usage of these APIs on unsupported platforms.

.NET Standard libraries can continue to use the latest available package
which harvests for .NET Standard configurations.

Contributes to #47530

To reduce complexity of the cryptography OpenSsl library, dead ending
its package as most of the configurations are partial facades anyway and
adding OpenSsl to the targeting pack. It's already part of the runtime
and exposed in aspnetcore's targeting pack.

.NET Standard libraries can continue to use the latest available package
which harvests for .NET Standard configurations.

Contributes to dotnet#47530
@ViktorHofer ViktorHofer added area-System.Security breaking-change Issue or PR that represents a breaking API or functional change over a prerelease. labels Apr 26, 2021
@ViktorHofer ViktorHofer added this to the 6.0.0 milestone Apr 26, 2021
@ViktorHofer ViktorHofer requested review from ericstj and bartonjs April 26, 2021 10:39
@ViktorHofer ViktorHofer self-assigned this Apr 26, 2021
@ghost
Copy link

ghost commented Apr 26, 2021

Tagging subscribers to this area: @bartonjs, @vcsjones, @krwq, @GrabYourPitchforks
See info in area-owners.md if you want to be subscribed.

Issue Details

To reduce complexity of the cryptography OpenSsl library, dead ending
its package as most of the configurations are partial facades anyway and
adding OpenSsl to the targeting pack. It's already part of the runtime
and exposed in aspnetcore's targeting pack.

.NET Standard libraries can continue to use the latest available package
which harvests for .NET Standard configurations.

Contributes to #47530

Author: ViktorHofer
Assignees: ViktorHofer
Labels:

area-System.Security, breaking-change

Milestone: 6.0.0

@ghost ghost added new-api-needs-documentation needs-breaking-change-doc-created Breaking changes need an issue opened with https://github.com/dotnet/docs/issues/new?template=dotnet labels Apr 26, 2021
@dotnet dotnet deleted a comment Apr 26, 2021
Copy link
Member

@ericstj ericstj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM modulo the cleanup that @bartonjs might want to do.

@ViktorHofer ViktorHofer merged commit 7b575d7 into dotnet:main Apr 26, 2021
@ViktorHofer ViktorHofer deleted the CryptoOpenSsl branch April 26, 2021 18:52
@ghost ghost locked as resolved and limited conversation to collaborators May 26, 2021
@ViktorHofer ViktorHofer removed the needs-breaking-change-doc-created Breaking changes need an issue opened with https://github.com/dotnet/docs/issues/new?template=dotnet label Sep 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-System.Security breaking-change Issue or PR that represents a breaking API or functional change over a prerelease.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants