Added Feature: CrpytoProvider-independent code#102
Closed
kraemv wants to merge 1 commit intorustls:mainfrom
Closed
Added Feature: CrpytoProvider-independent code#102kraemv wants to merge 1 commit intorustls:mainfrom
kraemv wants to merge 1 commit intorustls:mainfrom
Conversation
7a0c0a9 to
4a022f7
Compare
Added: New Entrypoint to build Added: New NIDs Added: Provider can be changed
4a022f7 to
d3e0e76
Compare
Member
I think it would be advantageous to split this off from the rest of the work in a separate PR. I don't have opinions on the rest yet, but suspect it will be a more involved discussion than adding a missing API surface. |
Contributor
Author
|
I will then create two separate PRs for the two features. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hello everyone,
I want to extend rustls-compat to work with variable CryptoProviders. Additionally I added the SSL_group_to_name entrypoint, as this is required by Debian 12 nginx.
The main changes are:
The main problem for switching the CryptoProvider in the old code was, that APIs outside the CryptoProvider API of rustls were used.
I am happy to discuss the changes!