Question

private static X509Certificate2 FindCertificate(string certificateSubject)
{
    const StoreName StoreName = StoreName.My;
    const StoreLocation StoreLocation = StoreLocation.LocalMachine;

    var store = new X509Store(StoreName, StoreLocation); 
    try
    {
        store.Open(OpenFlags.ReadOnly);

        // Find with the FindBySubjectName does fetch all the certs partially matching the subject name.
        // Hence, further filter for the certs that match the exact subject name.
        List<X509Certificate2> clientCertificates =
            store.Certificates.Find(X509FindType.FindBySubjectName, certificateSubject, validOnly: true)
                .Cast<X509Certificate2>()
                .Where(c => string.Equals(
                    c.Subject.Split(',').First().Trim(),
                    string.Concat("CN=", certificateSubject).Trim(),
                    StringComparison.OrdinalIgnoreCase)).ToList();

        if (!clientCertificates.Any())
        {
            throw new InvalidDataException(
                string.Format(CultureInfo.InvariantCulture, "Certificate {0} not found in the store {1}.", certificateSubject, StoreLocation.LocalMachine));
        }

        X509Certificate2 result = null;
        foreach (X509Certificate2 cert in clientCertificates)
        {
            DateTime now = DateTime.Now;
            DateTime effectiveDate = DateTime.Parse(cert.GetEffectiveDateString(), CultureInfo.CurrentCulture);
            DateTime expirationDate = DateTime.Parse(cert.GetExpirationDateString(), CultureInfo.CurrentCulture);
            if (effectiveDate <= now && expirationDate.Subtract(now) >= TimeSpan.FromDays(1))
            {
                result = cert;
                break;
            }
        }

        return result;
    }
    finally
    {
        store.Close();
    }
}

我的库中有这个代码，每次创建新请求时都会调用此方法。所以基本上每秒的请求数是1000，然后它将被调用1000次。当我使用PerfView工具时，我注意到此方法使用了35％的CPU。最大的罪魁祸首是store.Open和store.Certificates.Find方法。

其他人在他们的代码中发现了类似的问题。此外，如果您可以分享您为解决性能影响所做的工作。

Answer 1

只要目标系统没有安装绝大多数证书，您就可以跳过对X509Store的.Find()方法的调用。根据我的经验，它表现不佳，之后您已经为目标subjectName进行了必要的过滤。

另外，不要两次遍历X509Certificate2的集合！如果您只想要符合所有条件的第一个匹配证书，您可以简化为一个LINQ语句，如下所示：

X509Certificate2 cert =
    store.Certificates.Cast<X509Certificate2>()
        .FirstOrDefault(xc =>
            xc.Subject.Equals("CN=" + certificateSubject, StringComparison.OrdinalIgnoreCase)
            && xc.NotAfter >= DateTime.Now.AddDays(-1)
            && xc.NotBefore <= DateTime.Now);

（请注意，根据您的使用情况和证书，您可能需要也可能不需要修改上述内容，以便像原始代码一样用逗号分隔主题。）

最后，正如Wiktor Zychla所提到的，如果您的目标计算机没有安装大量证书，您可以通过调用store.Certificates.Cast<X509Certificate2>().ToList()来缓存整个证书列表，或者如果您的数量有限subjectNames。在您搜索的情况下，使用从subjectName派生的密钥和基于NotAfter属性的到期来简单地缓存此方法的结果可能更有效。

X509Store中store.Open和store.Certificates.Find方法导致的性能影响

1 个答案: