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.

Sign up for GitHub

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

Jump to bottom

[Test-Proxy] .NET Security Update Breaks Certificate Creation #6357

Closed
scbedd opened this issue Jun 16, 2023 · 1 comment
Closed

[Test-Proxy] .NET Security Update Breaks Certificate Creation #6357

scbedd opened this issue Jun 16, 2023 · 1 comment
Assignees
@scbedd
Labels
Test-Proxy Anything relating to test-proxy requests or issues.

Comments

@scbedd
Copy link
Member

scbedd commented Jun 16, 2023 •
edited
Loading

see this build

[xUnit.net 00:00:02.17]     Azure.Sdk.Tools.TestProxy.Tests.RecordingHandlerTests.TestSetRecordingOptionsValidTransportSessionLevel(body: "{{\"Transport\": {{\"Certificates\": [ {{ \"PemVal"...) [FAIL]
  Failed Azure.Sdk.Tools.TestProxy.Tests.RecordingHandlerTests.TestSetRecordingOptionsValidTransportSessionLevel(body: "{{\"Transport\": {{\"Certificates\": [ {{ \"PemVal"...) [87 ms]
  Error Message:
   Azure.Sdk.Tools.TestProxy.Common.Exceptions.HttpException : Unable to instantiate a new X509 certificate from the provided value and key. Failure Message: "PKCS12 (PFX) without a supplied password has exceeded maximum allowed iterations. See https://go.microsoft.com/fwlink/?linkid=2233907 for more information.".
  Stack Trace:
     at Azure.Sdk.Tools.TestProxy.RecordingHandler.GetTransport(Boolean allowAutoRedirect, TransportCustomizations customizations, Boolean insecure) in D:\a\_work\1\s\tools\test-proxy\Azure.Sdk.Tools.TestProxy\RecordingHandler.cs:line 717
   at Azure.Sdk.Tools.TestProxy.RecordingHandler.SetTransportOptions(TransportCustomizations customizations, String sessionId) in D:\a\_work\1\s\tools\test-proxy\Azure.Sdk.Tools.TestProxy\RecordingHandler.cs:line 797
   at Azure.Sdk.Tools.TestProxy.RecordingHandler.SetRecordingOptions(IDictionary`2 options, String sessionId) in D:\a\_work\1\s\tools\test-proxy\Azure.Sdk.Tools.TestProxy\RecordingHandler.cs:line 661
   at Azure.Sdk.Tools.TestProxy.Tests.RecordingHandlerTests.TestSetRecordingOptionsValidTransportSessionLevel(String body) in D:\a\_work\1\s\tools\test-proxy\Azure.Sdk.Tools.TestProxy.Tests\RecordingHandlerTests.cs:line 988

This is a new error that just started popping up. This is due to the fact that security update which enforces this rolled out on June 12th 2023. I believe it just now hit our agents.

Need to update the callsite, add an environment variable, or something, but this discussion link has some good methods to remediate.
@github-actions github-actions bot added the needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. label Jun 16, 2023
@scbedd scbedd self-assigned this Jun 16, 2023
@scbedd scbedd added Test-Proxy Anything relating to test-proxy requests or issues. and removed needs-triage Workflow: This is a new issue that needs to be triaged to the appropriate team. labels Jun 16, 2023
@scbedd scbedd mentioned this issue Jun 16, 2023
Merged
@scbedd
Copy link
Member Author

scbedd commented Jul 25, 2023

Resolved by #6358

@scbedd scbedd closed this as completed Jul 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@scbedd scbedd

Labels
Test-Proxy Anything relating to test-proxy requests or issues.
Projects
Azure SDK EngSys 📈
Archived in project
Milestone
No milestone
Development

No branches or pull requests
1 participant
@scbedd