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.

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

additional unit tests for SslProvider #2397

Closed
wants to merge 1 commit into from

Conversation

sullis
Copy link
Contributor

@sullis sullis commented Jul 23, 2022

No description provided.

@sullis sullis force-pushed the SslProviderTests-unit-tests branch from 565e56f to 2d22a20 Compare July 24, 2022 19:05
- new assertions confirm that SslProvider is OpenSSL
@sullis sullis force-pushed the SslProviderTests-unit-tests branch from 2d22a20 to 6a330ce Compare July 24, 2022 20:44
@violetagg
Copy link
Member

The failed test on CI is not relevant.

@violetagg violetagg added the type/enhancement A general enhancement label Jul 25, 2022
@violetagg violetagg added this to the 1.0.22 milestone Jul 25, 2022
violetagg pushed a commit that referenced this pull request Jul 25, 2022
- new assertions confirm that SslProvider is OpenSSL
violetagg added a commit that referenced this pull request Jul 25, 2022
violetagg added a commit that referenced this pull request Jul 25, 2022
@violetagg
Copy link
Member

@sullis Thanks for the PR!
I committed the changes to 1.0.x branch (1.0.22) with 69d63a0 then forward merged to main branch (1.1.0-M5) with 3a90eba and netty5 branch (2.0.0-M2) with 9893cda

@violetagg violetagg closed this Jul 25, 2022
@sullis sullis deleted the SslProviderTests-unit-tests branch December 21, 2022 04:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants