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

Add unit test that verifies we not fail hostname verification if a ma… #11803

Merged
merged 1 commit into from
Oct 28, 2021

Conversation

normanmaurer
Copy link
Member

…tching SNI name was used

Motivation:

We didn't have a unit test that verifies that the SNI hostname is respected when hostname verification is performed

Modifications:

Add unit test

Result:

Ensure we not regress

…tching SNI name was used

Motivation:

We didn't have a unit test that verifies that the SNI hostname is respected when hostname verification is performed

Modifications:

Add unit test

Result:

Ensure we not regress
@normanmaurer normanmaurer added this to the 4.1.70.Final milestone Oct 27, 2021
@normanmaurer normanmaurer merged commit 555fcf2 into 4.1 Oct 28, 2021
@normanmaurer normanmaurer deleted the hostname_verify_sni_test branch October 28, 2021 12:27
normanmaurer added a commit that referenced this pull request Oct 28, 2021
…tching SNI name was used (#11803)

Motivation:

We didn't have a unit test that verifies that the SNI hostname is respected when hostname verification is performed

Modifications:

Add unit test

Result:

Ensure we not regress
laosijikaichele pushed a commit to laosijikaichele/netty that referenced this pull request Dec 16, 2021
…tching SNI name was used (netty#11803)

Motivation:

We didn't have a unit test that verifies that the SNI hostname is respected when hostname verification is performed

Modifications:

Add unit test

Result:

Ensure we not regress
laosijikaichele pushed a commit to laosijikaichele/netty that referenced this pull request Dec 16, 2021
…tching SNI name was used (netty#11803)

Motivation:

We didn't have a unit test that verifies that the SNI hostname is respected when hostname verification is performed

Modifications:

Add unit test

Result:

Ensure we not regress
raidyue pushed a commit to raidyue/netty that referenced this pull request Jul 8, 2022
…tching SNI name was used (netty#11803)

Motivation:

We didn't have a unit test that verifies that the SNI hostname is respected when hostname verification is performed

Modifications:

Add unit test

Result:

Ensure we not regress
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant