Skip to content

Document WS-Federation remote sign-out controls - #69860

Open
Mr444-dev wants to merge 2 commits into
dotnet:mainfrom
Mr444-dev:docs/ws-federation-remote-sign-out
Open

Mr444-dev wants to merge 2 commits into
dotnet:mainfrom
Mr444-dev:docs/ws-federation-remote-sign-out

Conversation

@Mr444-dev

Copy link
Copy Markdown

Summary

Expands the WS-Federation API docs to explain that wsignoutcleanup1.0 requests aren't signed or session-bound, that requests reaching RemoteSignOutPath can trigger local sign-out, and how to disable the endpoint or filter requests in OnRemoteSignOut.

The event example rejects missing Referer headers and warns that request headers are metadata rather than cryptographic authentication. It also calls out provider-specific delivery and SameSite cookie behavior.

The companion tutorial update is in dotnet/AspNetCore.Docs#37907.

Addresses #69635.

Validation

  • git diff --check.
  • The targeted project build could not be completed locally: this checkout pins .NET SDK 11.0.100-rc.1.26420.103, while the machine has SDKs 7.0.401 and 10.0.401. The sparse checkout also lacks the generated build props from the repository restore step.

Copilot AI balanced review requested due to automatic review settings October 15, 2026 15:13

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions github-actions Bot added the area-auth Includes: authentication, authorization, OAuth, OIDC, and access token validation label Oct 15, 2026
@dotnet-policy-service dotnet-policy-service Bot added the community-contribution Indicates that the PR has been added by a community member label Oct 15, 2026
@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Thanks for your PR, @Mr444-dev. Someone from the team will get assigned to your PR shortly and we'll get it reviewed.

@Mr444-dev

Copy link
Copy Markdown
Author

@dotnet-policy-service agree

@dotnet-policy-service

Copy link
Copy Markdown
Contributor

Looks like this PR hasn't been active for some time and the codebase could have been changed in the meantime.
To make sure no conflicting changes have occurred, please rerun validation before merging. You can do this by leaving an /azp run comment here (requires commit rights), or by simply closing and reopening.

@dotnet-policy-service dotnet-policy-service Bot added the pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun label Oct 22, 2026
@Mr444-dev

Copy link
Copy Markdown
Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Commenter does not have sufficient privileges for PR 69860 in repo dotnet/aspnetcore

@Mr444-dev Mr444-dev closed this Oct 23, 2026
@Mr444-dev Mr444-dev reopened this Oct 23, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-auth Includes: authentication, authorization, OAuth, OIDC, and access token validation community-contribution Indicates that the PR has been added by a community member pending-ci-rerun When assigned to a PR indicates that the CI checks should be rerun

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants