Skip to content

Conversation

@wfurt
Copy link
Member

@wfurt wfurt commented Oct 28, 2021

add notes about SPN

followup on #26614

add notes about SPN
Copy link
Contributor

@gewarren gewarren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adding this!

| **runtimeconfig.json** | `System.Net.Http.UsePortInSpn` | `true` - includes port number in SPN e.g. `HTTP/host:port` <br>`false` - does not include port in SPN e.g. `HTTP/host`<br> |
| **Environment variable** | `DOTNET_SYSTEM_NET_HTTP_USEPORTINSPN` | `1` - includes port number in SPN e.g. `HTTP/host:port` <br/>`0` - does not include port in SPN e.g. `HTTP/host` |


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change

Copy link
Contributor

@gewarren gewarren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good - just one final suggestion.

Preview

@gewarren gewarren merged commit 3ff85d9 into main Oct 28, 2021
@gewarren gewarren deleted the wfurt-patch-1 branch October 28, 2021 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants