-
Notifications
You must be signed in to change notification settings - Fork 375
Fix obsolete X509Certificate2 Errors #14919
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
Conversation
It looks like we may have to pull in the latest from runtime in order for this to work at the repo level. |
👋 This is an interesting case as, currently, there is no replacement API for what you are using cc @bartonjs - this is another example of needing to suppress the obsoletion. |
See dotnet/docs#41662 (comment) for an example. We'd need to wrap it in I'd suggest just suppressing SYSLIB0057 for now. |
@vcsjones @akoeplinger - Thanks for your help! I appreciate it. I've updated the PR given the guidance |
Stage 2 build passed for arcade with these changes. I will have to investigate the arcade-pr failures after I finish with release work. |
Or add a package reference to Microsoft.Bcl.Cryptography (the back-compat package) |
05081fd
One of the tests was failing because the filePath did not contain a certificate. This resulted in GetCertContentType() throwing an exception. As a fix, I placed the check for the cert type in the try-catch block. |
Fixes dotnet/source-build#4494
Test build
My local build worked with thes changes, but I'm verifying with the source-build-lite pipeline as well.