Skip to content
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

[Feature]: Remove openssl submodule and code from CryptoPkg #674

Closed
spbrogan opened this issue Jan 13, 2024 · 0 comments · Fixed by #680
Closed

[Feature]: Remove openssl submodule and code from CryptoPkg #674

spbrogan opened this issue Jan 13, 2024 · 0 comments · Fixed by #680
Assignees
@kenlautner
Labels
complexity:intermediate Requires intermediate background information and effort to accomplish impact:breaking-change Requires integration attention type:feature-request A new feature proposal urgency:high Significant with a critical impact

Comments

@spbrogan
Copy link
Member

Feature Overview

The CryptoPkg supplies a provider agnostic api for crypto services and with the addition of the CryptoPkg/Driver infrastructure is enabled for binary inclusion of prepackaged crypto. Prepackaged crypto provides for better security, servicing, and developer productivity. To complete this story openssl and any future crypto providers need to be removed from the CryptoPkg.

Solution Overview

remove submodule
remove openssl specific files
remove basecryptlib, tlslib, hashlib implementations that use openssl

Move this content to https://github.com/microsoft/mu_crypto_release

Alternatives Considered

No response

Urgency

High

Are you going to implement the feature request?

I will implement the feature

Do you need maintainer feedback?

No maintainer feedback needed

Anything else?

No response
@spbrogan spbrogan added state:needs-triage Needs to triaged to determine next steps type:feature-request A new feature proposal labels Jan 13, 2024
@github-actions github-actions bot added the urgency:high Significant with a critical impact label Jan 13, 2024
@spbrogan spbrogan self-assigned this Jan 13, 2024
@spbrogan spbrogan added impact:breaking-change Requires integration attention complexity:intermediate Requires intermediate background information and effort to accomplish and removed state:needs-triage Needs to triaged to determine next steps labels Jan 13, 2024
@kenlautner kenlautner assigned kenlautner and unassigned spbrogan Jan 16, 2024
@kenlautner kenlautner linked a pull request Jan 17, 2024 that will close this issue
Remove the openssl library and it's associated files from CryptoPkg. #680
Merged
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kenlautner kenlautner

Labels
complexity:intermediate Requires intermediate background information and effort to accomplish impact:breaking-change Requires integration attention type:feature-request A new feature proposal urgency:high Significant with a critical impact
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove the openssl library and it's associated files from CryptoPkg. microsoft/mu_basecore
2 participants
@spbrogan @kenlautner