Skip to content

Merge main into live#12733

Open
dotnet-policy-service[bot] wants to merge 3 commits into
livefrom
main
Open

Merge main into live#12733
dotnet-policy-service[bot] wants to merge 3 commits into
livefrom
main

Conversation

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Please don't squash-merge this PR.

* Initial plan

* Remove outdated DeflateStream 4 GB limit remarks

* Remove empty constructor remarks block in DeflateStream docs

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
…#12724)

* Clarify NegotiateAuthentication.IsAuthenticated post-failure behavior

The IsAuthenticated property returns true after the authentication exchange has finished, including in many post-negotiation failure cases (verified across the Windows, Unix, and ManagedSpnego PAL implementations in dotnet/runtime). The previous summary (successfully completed and the session was established) implied that a true value means successful authentication, which is misleading.

Update the summary, value, and remarks to clarify that callers must inspect the NegotiateAuthenticationStatusCode returned by GetOutgoingBlob to determine actual success.

Also remove a misleading `(see IsAuthenticated)` reference on TargetName that reinforced the same misconception.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* Potential fix for pull request finding

Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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.

3 participants