-
Notifications
You must be signed in to change notification settings - Fork 3
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
fix: only fetch valid X.509 certificates from the certificate store #425
Conversation
perf: remove linq code use to improve code generation
I'd like a 2nd set of eyes on the certificate scanning changes cc @baywet, @andrueastman |
src/Microsoft.Graph.Cli.Core/Authentication/ClientCertificateCredentialFactory.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.Graph.Cli.Core/Authentication/ClientCertificateCredentialFactory.cs
Outdated
Show resolved
Hide resolved
I'm adding some tests to improve coverage. |
Is it possible to unit test certificates? |
You'd need to generate self-signed certificates as part of the tests setup and add them to the user store. (machine store requires elevation) |
chore: remove redundant certificate date checks
src/Microsoft.Graph.Cli.Core/Authentication/ClientCertificateCredentialFactory.cs
Show resolved
Hide resolved
…redentialFactory.cs Co-authored-by: Vincent Biret <vibiret@microsoft.com>
Quality Gate failedFailed conditions |
Not sure if I can improve coverage meaningfully anymore... :-/ |
perf: remove linq code use to improve code generation