Skip to content

[release/8.0] Skip NegotiateStream_StreamToStream_Authentication_EmptyCredentials_Fails on WinSrv 2025 - #112472

Merged
rzikm merged 1 commit into
dotnet:release/8.0-stagingfrom
rzikm:win2025-empty-creds-8.0
Feb 13, 2025
Merged

[release/8.0] Skip NegotiateStream_StreamToStream_Authentication_EmptyCredentials_Fails on WinSrv 2025#112472
rzikm merged 1 commit into
dotnet:release/8.0-stagingfrom
rzikm:win2025-empty-creds-8.0

Conversation

@rzikm

Copy link
Copy Markdown
Member

Manual port of 92403f5 which did not made it in release branches.

Fixes#112312.

Customer Impact

None, Test-only change

Regression

No, new test failure occurred after adding new platform to test matrix.

Testing

CI run succeeds

Risk

Low, test-only change.

@wfurtwfurt left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LFTM

@rzikm

Copy link
Copy Markdown
MemberAuthor

Test-only change -> tell mode

@rzikmrzikm added the Servicing-approved Approved for servicing release label Feb 13, 2025
@rzikm

Copy link
Copy Markdown
MemberAuthor

/ba-g WinHttpHandler failure is unrelated

@rzikm
rzikm merged commit 6f7daef into dotnet:release/8.0-stagingFeb 13, 2025
{
string targetName = "testTargetName";

if (PlatformDetection.IsWindowsServer2025)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is a misnomer - it's checking any windows SKU newer than build 26100 -- so you've effectively turned this off these tests on all new windows machines. I noticed as this was failing in release branches where we haven't added testing for Server 2025.

@github-actionsgithub-actionsBot locked and limited conversation to collaborators Mar 22, 2025
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Net.SecurityServicing-approvedApproved for servicing release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@rzikm@ericstj@wfurt