fix: catch PackagesConfigReaderException in NuGetPackagesConfigDetector (#362)

Author: JamieMagee

src/Microsoft.ComponentDetection.Detectors/nuget/NuGetPackagesConfigDetector.cs

@@ -26,16 +26,23 @@ public class NuGetPackagesConfigDetector : FileComponentDetector
         protected override Task OnFileFound(ProcessRequest processRequest, IDictionary<string, string> detectorArgs)
         {
             var packagesConfig = new PackagesConfigReader(processRequest.ComponentStream.Stream);
-            foreach (var package in packagesConfig.GetPackages(allowDuplicatePackageIds: true))
+            try
             {
-                processRequest.SingleFileComponentRecorder.RegisterUsage(
-                    new DetectedComponent(
-                        new NuGetComponent(
-                            package.PackageIdentity.Id,
-                            package.PackageIdentity.Version.ToNormalizedString())),
-                    true,
-                    null,
-                    package.IsDevelopmentDependency);
+                foreach (var package in packagesConfig.GetPackages(allowDuplicatePackageIds: true))
+                {
+                    processRequest.SingleFileComponentRecorder.RegisterUsage(
+                        new DetectedComponent(
+                            new NuGetComponent(
+                                package.PackageIdentity.Id,
+                                package.PackageIdentity.Version.ToNormalizedString())),
+                        true,
+                        null,
+                        package.IsDevelopmentDependency);
+                }
+            }
+            catch (PackagesConfigReaderException e)
+            {
+                this.Logger.LogFailedReadingFile(processRequest.ComponentStream.Location, e);
             }
 
             return Task.CompletedTask;

test/Microsoft.ComponentDetection.VerificationTests/resources/nuget/packages-config/packages.config

@@ -1,5 +1,5 @@
-<!-- https://github.com/dotnet/Kerberos.NET/blob/559bec87d6eb1fb8698418c3aacafbbc7c32bdec/Samples/KerberosWebSample/packages.config -->
-<?xml version="1.0" encoding="utf-8"?>
+<?xml version="1.0" encoding="utf-8"?>
+<!-- https://github.com/dotnet/Kerberos.NET/blob/559bec87d6eb1fb8698418c3aacafbbc7c32bdec/Samples/KerberosWebSample/packages.config -->
 <packages>
   <package id="Microsoft.AspNet.WebApi.Client" version="5.2.3" targetFramework="net452" />
   <package id="Microsoft.AspNet.WebApi.Core" version="5.2.3" targetFramework="net452" />