Fix snippets

[maa]

Fix snippets

[maa]

@maa please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.

@dotnet-policy-service agree [company="{your company}"]

Options:

• (default - no company specified) I have sole ownership of intellectual property rights to my Submissions and I am not making Submissions in the course of work for my employer.

@dotnet-policy-service agree

• (when company given) I am making Submissions in the course of work for my employer (or my employer has intellectual property rights in my Submissions by contract or applicable law). I have permission from my employer to make Submissions and enter into this Agreement on behalf of my employer. By signing below, the defined term “You” includes me and my employer.

@dotnet-policy-service agree company="Microsoft"

Contributor License Agreement

@dotnet-policy-service agree

[guardrex]

Hello @maa ... Per our contribution guidelines, we usually require an issue (opened from the bottom of the article using the Open a documentation issue link) to discuss changes before a PR is submitted. We recommend it for discussion and tracking the work done on the repo. However, we do accept PRs for small bug fixes.

You didn't say in your OP ☝️ what the "fix" is. The diff is a bit hard to read on this one, but it looks like you're moving the type declarations for localhostCert and exampleCert. Is that all this is doing, and why are you moving them?

Also note that you must agree to the contributor license agreement (if you agree with its terms) for us to accept PRs from you.

[maa]

Hello @guardrex.

The diff is a bit hard to read on this one, but it looks like you're moving the type declarations for localhostCert and exampleCert. Is that all this is doing, and why are you moving them?

Nope. I think current documentation have incorrect usage of 'listenOptions.UseHttps' in sections 'SNI with ServerOptionsSelectionCallback' and 'SNI with TlsHandshakeCallbackOptions'. 'UseHttps' method called twice, one inside another, which seems to me a little bit wrong.

[guardrex]

I see! It was a bit hard to see the way that GH displayed the diff.

@Tratcher ... You placed these code snippets with the nested UseHttps calls on #23302. Would you take a look at the changes that @maa is suggesting?