2 
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 方法。

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

4

1 回答 1

4

只要目标系统没有安装大量证书，您就可以跳过对 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属性的过期来简单地缓存此方法的结果。

于 2015-11-05T21:42:28.880 回答