Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

System.Security.Cryptography.X509Certificates.Tests.ChainTests.BuildInvalidSignatureTwice failure #82837

Open
steveisok opened this issue Mar 1, 2023 · 6 comments · Fixed by #83542
Labels
area-System.Security disabled-test The test is disabled in source code against the issue Known Build Error Use this to report build issues in the .NET Helix tab runtime-mono specific to the Mono runtime
Milestone

Comments

@steveisok
Copy link
Member

steveisok commented Mar 1, 2023

Build Information

Build: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=189217

Build error leg or test failing: System.Net.Security.Tests.CertificateValidationClientServer.CertificateValidationClientServer_EndToEnd_Ok

Pull request: #82826

System.Security.Cryptography.X509Certificates.Tests.ChainTests.BuildInvalidSignatureTwice [FAIL]
    Chain is valid on execution 1
    Expected: False
    Actual:   True
    Stack Trace:
    /_/src/libraries/System.Security.Cryptography.X509Certificates/tests/ChainTests.cs(1019,0): at System.Security.Cryptography.X509Certificates.Tests.ChainTests.<BuildInvalidSignatureTwice>g__CheckChain|28_0(<>c__DisplayClass28_0&, <>c__DisplayClass28_1&, <>c__DisplayClass28_2&)
    /_/src/libraries/System.Security.Cryptography.X509Certificates/tests/ChainTests.cs(1025,0): at System.Security.Cryptography.X509Certificates.Tests.ChainTests.BuildInvalidSignatureTwice()

Error Message

{
  "ErrorMessage": "Chain is valid on execution 1",
  "BuildRetry": false,
  "ErrorPattern": "",
  "ExcludeConsoleLog": false
}

Report

Build Definition Test Pull Request
878306 dotnet/runtime System.Security.Cryptography.Tests.WorkItemExecution
869750 dotnet/runtime System.Security.Cryptography.X509Certificates.Tests.ChainTests.BuildInvalidSignatureTwice #109818
867413 dotnet/runtime System.Security.Cryptography.Tests.WorkItemExecution

Summary

24-Hour Hit Count 7-Day Hit Count 1-Month Count
0 1 3
@steveisok steveisok added blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' Known Build Error Use this to report build issues in the .NET Helix tab labels Mar 1, 2023
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Mar 1, 2023
@ghost
Copy link

ghost commented Mar 1, 2023

Tagging subscribers to this area: @dotnet/area-system-security, @vcsjones
See info in area-owners.md if you want to be subscribed.

Issue Details

Build Information

Build: https://dev.azure.com/dnceng-public/cbb18261-c48f-4abb-8651-8cdcb5474649/_build/results?buildId=189217
Build error leg or test failing: System.Net.Security.Tests.CertificateValidationClientServer.CertificateValidationClientServer_EndToEnd_OkPull request: #82826

System.Security.Cryptography.X509Certificates.Tests.ChainTests.BuildInvalidSignatureTwice [FAIL]
    Chain is valid on execution 1
    Expected: False
    Actual:   True
    Stack Trace:
    /_/src/libraries/System.Security.Cryptography.X509Certificates/tests/ChainTests.cs(1019,0): at System.Security.Cryptography.X509Certificates.Tests.ChainTests.<BuildInvalidSignatureTwice>g__CheckChain|28_0(<>c__DisplayClass28_0&, <>c__DisplayClass28_1&, <>c__DisplayClass28_2&)
    /_/src/libraries/System.Security.Cryptography.X509Certificates/tests/ChainTests.cs(1025,0): at System.Security.Cryptography.X509Certificates.Tests.ChainTests.BuildInvalidSignatureTwice()

Error Message

{
  "ErrorMessage": "Chain is valid on execution 1",
  "BuildRetry": false,
  "ErrorPattern": "",
  "ExcludeConsoleLog": false
}
Author: steveisok
Assignees: -
Labels:

area-System.Security, blocking-clean-ci, Known Build Error

Milestone: -

@steveisok
Copy link
Member Author

/cc @jeffhandley

@vcsjones
Copy link
Member

vcsjones commented Mar 1, 2023

Am I correct in observing that this test only fails in linux-Debug-x64-mono_interpreter_release?

@steveisok
Copy link
Member Author

Yes, as of right now it appears that way.

@vcsjones vcsjones added disabled-test The test is disabled in source code against the issue and removed blocking-clean-ci Blocking PR or rolling runs of 'runtime' or 'runtime-extra-platforms' Known Build Error Use this to report build issues in the .NET Helix tab labels Mar 2, 2023
@jeffhandley jeffhandley added this to the 8.0.0 milestone Mar 8, 2023
@ghost ghost removed the untriaged New issue has not been triaged by the area owner label Mar 8, 2023
@jeffhandley jeffhandley added the needs-further-triage Issue has been initially triaged, but needs deeper consideration or reconsideration label Mar 8, 2023
@ghost ghost added the in-pr There is an active PR which will close this issue when it is merged label Mar 16, 2023
@ghost ghost removed the in-pr There is an active PR which will close this issue when it is merged label Mar 17, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Apr 16, 2023
@hoyosjs
Copy link
Member

hoyosjs commented May 9, 2023

Enabling again - seen in CI

@hoyosjs hoyosjs reopened this May 9, 2023
@hoyosjs hoyosjs added the Known Build Error Use this to report build issues in the .NET Helix tab label May 9, 2023
@adamsitnik
Copy link
Member

Am I correct in observing that this test only fails in linux-Debug-x64-mono_interpreter_release?

@steveisok If this is the case then we need some help from Mono Team.

@build-analysis build-analysis bot removed this from the 9.0.0 milestone Nov 15, 2023
@ghost ghost added the untriaged New issue has not been triaged by the area owner label Nov 15, 2023
@jeffhandley jeffhandley added this to the 9.0.0 milestone Nov 17, 2023
@ghost ghost removed the untriaged New issue has not been triaged by the area owner label Nov 17, 2023
@dotnet dotnet unlocked this conversation Nov 24, 2023
@jeffhandley jeffhandley removed the needs-further-triage Issue has been initially triaged, but needs deeper consideration or reconsideration label Aug 6, 2024
@jeffhandley jeffhandley modified the milestones: 9.0.0, Future Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-System.Security disabled-test The test is disabled in source code against the issue Known Build Error Use this to report build issues in the .NET Helix tab runtime-mono specific to the Mono runtime
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants