fix 2 warnings in CodeAnalysis (#3388)

* fix 2 warnings in CodeAnalysis
NuGet · Jun 16, 2020 · aa49e4d · aa49e4d
1 parent 146812c
commit aa49e4d
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/NuGet.Core/NuGet.Packaging/Signing/Cms/ManagedCmsWrapper.cs b/src/NuGet.Core/NuGet.Packaging/Signing/Cms/ManagedCmsWrapper.cs
@@ -47,7 +47,7 @@ public void AddCertificates(IEnumerable<X509Certificate2> certificates)
         {
             foreach (var cert in certificates)
             {
-                _signedCms.AddCertificate(new X509Certificate2(cert));
+                    _signedCms.AddCertificate(cert);
             }
         }
 

diff --git a/src/NuGet.Core/NuGet.Packaging/Signing/Timestamp/Rfc3161TimestampProvider.cs b/src/NuGet.Core/NuGet.Packaging/Signing/Timestamp/Rfc3161TimestampProvider.cs
@@ -26,8 +26,9 @@ public class Rfc3161TimestampProvider : ITimestampProvider
     {
         // Url to an RFC 3161 timestamp server
         private readonly Uri _timestamperUrl;
+#if IS_SIGNING_SUPPORTED
         private static readonly TimeSpan RequestTimeout = TimeSpan.FromSeconds(10);
-
+#endif
         public Rfc3161TimestampProvider(Uri timeStampServerUrl)
         {
 #if IS_SIGNING_SUPPORTED
-Original file line number
+Diff line change
@@ Expand Up @@
             {
                 foreach (var cert in certificates)
                 {
-                    _signedCms.AddCertificate(new X509Certificate2(cert));
+                        _signedCms.AddCertificate(cert);
                 }
             }
@@ Expand Down @@