Reapply "Update packageurl-dotnet to 2.0.0-rc.2 (#1730)" (#1751) (#1753)

JamieMagee · web-flow · commit 7fb976c3f078 · 2026-03-31T17:36:42.000+01:00
diff --git a/Directory.Packages.props b/Directory.Packages.props
@@ -27,7 +27,7 @@
     <PackageVersion Include="Newtonsoft.Json.Schema" Version="4.0.1" />
     <PackageVersion Include="NuGet.ProjectModel" Version="7.3.0" />
     <PackageVersion Include="NuGet.Versioning" Version="7.3.0" />
-    <PackageVersion Include="packageurl-dotnet" Version="1.0.0" />
+    <PackageVersion Include="packageurl-dotnet" Version="2.0.0-rc.3" />
     <PackageVersion Include="Polly" Version="8.6.6" />
     <PackageVersion Include="SemanticVersioning" Version="2.0.2" />
     <PackageVersion Include="Serilog" Version="4.3.1" />
diff --git a/docs/creating-a-new-detector.md b/docs/creating-a-new-detector.md
@@ -104,7 +104,7 @@ public class YourEcosystemComponent : TypedComponent
     public string Version { get; set; }
 
     public override ComponentType Type => ComponentType.YourType;
-    public override PackageURL PackageUrl => new PackageURL("your-type", null, this.Name, this.Version, null, null);
+    public override PackageUrl PackageUrl => new PackageUrl("your-type", null, this.Name, this.Version, null, null);
     protected override string ComputeId() => $"{this.Name} {this.Version} - {this.Type}";
 }
 ```
diff --git a/docs/schema/manifest.schema.json b/docs/schema/manifest.schema.json
@@ -547,4 +547,4 @@
     "resultCode",
     "sourceDirectory"
   ]
-}
+}
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/CargoComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/CargoComponent.cs
@@ -44,7 +44,7 @@ public CargoComponent(string name, string version, string author = null, string
     public override ComponentType Type => ComponentType.Cargo;
 
     [JsonPropertyName("packageUrl")]
-    public override PackageURL PackageUrl => new PackageURL("cargo", string.Empty, this.Name, this.Version, null, string.Empty);
+    public override PackageUrl PackageUrl => new PackageUrl("cargo", string.Empty, this.Name, this.Version, null, string.Empty);
 
     protected override string ComputeBaseId() => $"{this.Name} {this.Version} - {this.Type}";
 }
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/ConanComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/ConanComponent.cs
@@ -38,7 +38,7 @@ public ConanComponent(string name, string version, string previous, string packa
     public override ComponentType Type => ComponentType.Conan;
 
     [JsonPropertyName("packageUrl")]
-    public override PackageURL PackageUrl => new PackageURL("conan", string.Empty, this.Name, this.Version, null, string.Empty);
+    public override PackageUrl PackageUrl => new PackageUrl("conan", string.Empty, this.Name, this.Version, null, string.Empty);
 
     protected override string ComputeBaseId() => $"{this.Name} {this.Version} - {this.Type}";
 }
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/CppSdkComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/CppSdkComponent.cs
@@ -39,15 +39,15 @@ public CppSdkComponent(string name, string version)
     public override ComponentType Type => ComponentType.CppSdk;
 
     [JsonPropertyName("packageUrl")]
-    public override PackageURL PackageUrl
+    public override PackageUrl PackageUrl
     {
         get
         {
             var qualifiers = new SortedDictionary<string, string>
             {
                 { "type", "cppsdk" },
             };
-            return new PackageURL("generic", null, this.Name, this.Version, qualifiers, null);
+            return new PackageUrl("generic", null, this.Name, this.Version, qualifiers, null);
         }
     }
 
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/GoComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/GoComponent.cs
@@ -37,14 +37,34 @@ public GoComponent()
 
     // Commit should be used in place of version when available
     // https://github.com/package-url/purl-spec/blame/180c46d266c45aa2bd81a2038af3f78e87bb4a25/README.rst#L610
+    // The golang purl spec requires a namespace: https://github.com/package-url/purl-spec/blob/master/types/golang-definition.json
     [JsonPropertyName("packageUrl")]
-    public override PackageURL PackageUrl => new PackageURL("golang", null, this.Name, string.IsNullOrWhiteSpace(this.Hash) ? this.Version : this.Hash, null, null);
+    public override PackageUrl PackageUrl
+    {
+        get
+        {
+            var version = string.IsNullOrWhiteSpace(this.Hash) ? this.Version : this.Hash;
+            var (ns, name) = this.GetNamespaceAndName();
+            return new PackageUrl("golang", ns, name, version, null, null);
+        }
+    }
 
     [JsonIgnore]
     public override ComponentType Type => ComponentType.Go;
 
     protected override string ComputeBaseId() => $"{this.Name} {this.Version} - {this.Type}";
 
+    private (string Namespace, string Name) GetNamespaceAndName()
+    {
+        var lastSlash = this.Name.LastIndexOf('/');
+        if (lastSlash > 0)
+        {
+            return (this.Name.Substring(0, lastSlash), this.Name.Substring(lastSlash + 1));
+        }
+
+        return (null, this.Name);
+    }
+
     public override bool Equals(object obj)
     {
         return obj is GoComponent otherComponent && this.Equals(otherComponent);
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/LinuxComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/LinuxComponent.cs
@@ -46,7 +46,7 @@ public LinuxComponent(string distribution, string release, string name, string v
     public override ComponentType Type => ComponentType.Linux;
 
     [JsonPropertyName("packageUrl")]
-    public override PackageURL PackageUrl
+    public override PackageUrl PackageUrl
     {
         get
         {
@@ -63,7 +63,7 @@ public override PackageURL PackageUrl
 
             if (packageType != null)
             {
-                return new PackageURL(packageType, this.Distribution, this.Name, this.Version, null, null);
+                return new PackageUrl(packageType, this.Distribution, this.Name, this.Version, null, null);
             }
 
             return null;
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/MavenComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/MavenComponent.cs
@@ -31,7 +31,7 @@ public MavenComponent()
     public override ComponentType Type => ComponentType.Maven;
 
     [JsonPropertyName("packageUrl")]
-    public override PackageURL PackageUrl => new PackageURL("maven", this.GroupId, this.ArtifactId, this.Version, null, null);
+    public override PackageUrl PackageUrl => new PackageUrl("maven", this.GroupId, this.ArtifactId, this.Version, null, null);
 
     protected override string ComputeBaseId() => $"{this.GroupId} {this.ArtifactId} {this.Version} - {this.Type}";
 }
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/NpmComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/NpmComponent.cs
@@ -36,7 +36,7 @@ public NpmComponent(string name, string version, string hash = null, NpmAuthor a
     public override ComponentType Type => ComponentType.Npm;
 
     [JsonPropertyName("packageUrl")]
-    public override PackageURL PackageUrl => new PackageURL("npm", null, this.Name, this.Version, null, null);
+    public override PackageUrl PackageUrl => new PackageUrl("npm", null, this.Name, this.Version, null, null);
 
     protected override string ComputeBaseId() => $"{this.Name} {this.Version} - {this.Type}";
 }
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/NugetComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/NugetComponent.cs
@@ -31,7 +31,7 @@ public NuGetComponent(string name, string version, string[] authors = null)
     public override ComponentType Type => ComponentType.NuGet;
 
     [JsonPropertyName("packageUrl")]
-    public override PackageURL PackageUrl => new PackageURL("nuget", null, this.Name, this.Version, null, null);
+    public override PackageUrl PackageUrl => new PackageUrl("nuget", null, this.Name, this.Version, null, null);
 
     protected override string ComputeBaseId() => $"{this.Name} {this.Version} - {this.Type}";
 }
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/PipComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/PipComponent.cs
@@ -40,7 +40,7 @@ public PipComponent(string name, string version, string author = null, string li
     public override ComponentType Type => ComponentType.Pip;
 
     [JsonPropertyName("packageUrl")]
-    public override PackageURL PackageUrl => new PackageURL("pypi", null, this.Name, this.Version, null, null);
+    public override PackageUrl PackageUrl => new PackageUrl("pypi", null, this.Name, this.Version, null, null);
 
     [SuppressMessage("Usage", "CA1308:Normalize String to Uppercase", Justification = "Casing cannot be overwritten.")]
     protected override string ComputeBaseId() => $"{this.Name} {this.Version} - {this.Type}".ToLowerInvariant();
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/PodComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/PodComponent.cs
@@ -32,7 +32,7 @@ public PodComponent(string name, string version, string specRepo = "")
     public override ComponentType Type => ComponentType.Pod;
 
     [JsonPropertyName("packageUrl")]
-    public override PackageURL PackageUrl
+    public override PackageUrl PackageUrl
     {
         get
         {
@@ -42,7 +42,7 @@ public override PackageURL PackageUrl
                 qualifiers.Add("repository_url", this.SpecRepo);
             }
 
-            return new PackageURL("cocoapods", null, this.Name, this.Version, qualifiers, null);
+            return new PackageUrl("cocoapods", null, this.Name, this.Version, qualifiers, null);
         }
     }
 
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/RubyGemsComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/RubyGemsComponent.cs
@@ -31,7 +31,7 @@ public RubyGemsComponent(string name, string version, string source = "")
     public override ComponentType Type => ComponentType.RubyGems;
 
     [JsonPropertyName("packageUrl")]
-    public override PackageURL PackageUrl => new PackageURL("gem", null, this.Name, this.Version, null, null);
+    public override PackageUrl PackageUrl => new PackageUrl("gem", null, this.Name, this.Version, null, null);
 
     protected override string ComputeBaseId() => $"{this.Name} {this.Version} - {this.Type}";
 }
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/SwiftComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/SwiftComponent.cs
@@ -40,12 +40,12 @@ public SwiftComponent(string name, string version, string packageUrl, string has
     [JsonIgnore]
     public override ComponentType Type => ComponentType.Swift;
 
-    // Example PackageURL -> pkg:swift/github.com/apple/swift-asn1
+    // Example PackageUrl -> pkg:swift/github.com/apple/swift-asn1
     // type: swift
     // namespace: github.com/apple
     // name: swift-asn1
     [JsonPropertyName("packageUrl")]
-    public override PackageURL PackageUrl => new PackageURL(
+    public override PackageUrl PackageUrl => new PackageUrl(
         type: "swift",
         @namespace: this.GetNamespaceFromPackageUrl(),
         name: this.Name,
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/TypedComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/TypedComponent.cs
@@ -61,7 +61,7 @@ internal TypedComponent()
     public string BaseId => this.baseId ??= this.ComputeBaseId();
 
     [SystemTextJson.JsonPropertyName("packageUrl")]
-    public virtual PackageURL PackageUrl { get; }
+    public virtual PackageUrl PackageUrl { get; }
 
     /// <summary>Gets or sets SPDX license expression(s) declared by the package author.</summary>
     [JsonProperty("licenses", NullValueHandling = NullValueHandling.Ignore)]
diff --git a/src/Microsoft.ComponentDetection.Contracts/TypedComponent/VcpkgComponent.cs b/src/Microsoft.ComponentDetection.Contracts/TypedComponent/VcpkgComponent.cs
@@ -49,21 +49,21 @@ public VcpkgComponent(string spdxid, string name, string version, string triplet
     public override ComponentType Type => ComponentType.Vcpkg;
 
     [JsonPropertyName("packageUrl")]
-    public override PackageURL PackageUrl
+    public override PackageUrl PackageUrl
     {
         get
         {
             if (this.PortVersion > 0)
             {
-                return new PackageURL($"pkg:vcpkg/{this.Name}@{this.Version}?port_version={this.PortVersion}");
+                return new PackageUrl($"pkg:vcpkg/{this.Name}@{this.Version}?port_version={this.PortVersion}");
             }
             else if (this.Version != null)
             {
-                return new PackageURL($"pkg:vcpkg/{this.Name}@{this.Version}");
+                return new PackageUrl($"pkg:vcpkg/{this.Name}@{this.Version}");
             }
             else
             {
-                return new PackageURL($"pkg:vcpkg/{this.Name}");
+                return new PackageUrl($"pkg:vcpkg/{this.Name}");
             }
         }
     }
diff --git a/test/Microsoft.ComponentDetection.Contracts.Tests/CppSdkComponentTests.cs b/test/Microsoft.ComponentDetection.Contracts.Tests/CppSdkComponentTests.cs
@@ -77,9 +77,7 @@ public void PackageUrl_ShouldReturnCorrectFormat()
         var packageUrl = component.PackageUrl;
 
         packageUrl.Type.Should().Be("generic");
-#pragma warning disable CA1308 // PackageURL normalizes to lowercase
-        packageUrl.Name.Should().Be(name.ToLowerInvariant());
-#pragma warning restore CA1308
+        packageUrl.Name.Should().Be(name);
         packageUrl.Version.Should().Be(version);
         packageUrl.Namespace.Should().BeNull();
         packageUrl.Qualifiers.Should().ContainKey("type");
diff --git a/test/Microsoft.ComponentDetection.Contracts.Tests/PurlGenerationTests.cs b/test/Microsoft.ComponentDetection.Contracts.Tests/PurlGenerationTests.cs
@@ -24,7 +24,7 @@ public void GoPackageShouldPreferHashOverVersion()
     {
         // Commit should be used in place of version when available
         // https://github.com/package-url/purl-spec/blame/180c46d266c45aa2bd81a2038af3f78e87bb4a25/README.rst#L610
-        var goComponent = new GoComponent("test", "1.2.3", "deadbeef");
+        var goComponent = new GoComponent("github.com/example/test", "1.2.3", "deadbeef");
         goComponent.PackageUrl.Version.Should().Be("deadbeef");
     }
 
@@ -97,9 +97,9 @@ public void CocoaPodNameShouldSupportPurl()
         var packageThree = new PodComponent("googleUtilities", "7.5.2");
 
         packageOne.PackageUrl.Type.Should().Be("cocoapods");
-        packageOne.PackageUrl.ToString().Should().Be("pkg:cocoapods/afnetworking@4.0.1");
-        packageTwo.PackageUrl.ToString().Should().Be("pkg:cocoapods/mapsindoors@3.24.0");
-        packageThree.PackageUrl.ToString().Should().Be("pkg:cocoapods/googleutilities@7.5.2");
+        packageOne.PackageUrl.ToString().Should().Be("pkg:cocoapods/AFNetworking@4.0.1");
+        packageTwo.PackageUrl.ToString().Should().Be("pkg:cocoapods/MapsIndoors@3.24.0");
+        packageThree.PackageUrl.ToString().Should().Be("pkg:cocoapods/googleUtilities@7.5.2");
     }
 
     [TestMethod]
@@ -108,6 +108,6 @@ public void CocoaPodNameShouldPurlWithCustomQualifier()
         // https://github.com/package-url/purl-spec/blob/b8ddd39a6d533b8895f3b741f2e62e2695d82aa4/PURL-TYPES.rst#cocoapods
         var packageOne = new PodComponent("AFNetworking", "4.0.1", "https://custom_repo.example.com/path/to/repo/specs.git");
 
-        packageOne.PackageUrl.ToString().Should().Be("pkg:cocoapods/afnetworking@4.0.1?repository_url=https://custom_repo.example.com/path/to/repo/specs.git");
+        packageOne.PackageUrl.ToString().Should().Be("pkg:cocoapods/AFNetworking@4.0.1?repository_url=https:%2F%2Fcustom_repo.example.com%2Fpath%2Fto%2Frepo%2Fspecs.git");
     }
 }
diff --git a/test/Microsoft.ComponentDetection.Contracts.Tests/TypedComponentSerializationTests.cs b/test/Microsoft.ComponentDetection.Contracts.Tests/TypedComponentSerializationTests.cs
@@ -162,12 +162,12 @@ public void TypedComponent_Serialization_Pip()
     [TestMethod]
     public void TypedComponent_Serialization_Go()
     {
-        TypedComponent tc = new GoComponent("SomeGoPackage", "1.2.3", "SomeHash");
+        TypedComponent tc = new GoComponent("github.com/example/SomeGoPackage", "1.2.3", "SomeHash");
         var result = JsonSerializer.Serialize(tc);
         var deserializedTC = JsonSerializer.Deserialize<TypedComponent>(result);
         deserializedTC.Should().BeOfType(typeof(GoComponent));
         var goComponent = (GoComponent)deserializedTC;
-        goComponent.Name.Should().Be("SomeGoPackage");
+        goComponent.Name.Should().Be("github.com/example/SomeGoPackage");
         goComponent.Version.Should().Be("1.2.3");
         goComponent.Hash.Should().Be("SomeHash");
     }
@@ -305,7 +305,7 @@ public void TypedComponent_Serialization_AllComponentTypes_TypePropertyNotDuplic
             new NuGetComponent("test", "1.0.0"),
             new MavenComponent("group", "artifact", "1.0.0"),
             new PipComponent("test", "1.0.0"),
-            new GoComponent("test", "1.0.0"),
+            new GoComponent("github.com/example/test", "1.0.0"),
             new CargoComponent("test", "1.0.0"),
             new RubyGemsComponent("test", "1.0.0"),
             new GitComponent(new Uri("https://github.com/test/test"), "abc123"),
diff --git a/test/Microsoft.ComponentDetection.Detectors.Tests/SwiftComponentTests.cs b/test/Microsoft.ComponentDetection.Detectors.Tests/SwiftComponentTests.cs
@@ -102,7 +102,7 @@ public void PackageURL_ShouldReturnCorrectPackageURL_GithubHostname()
 
         var component = new SwiftComponent(name, version, packageUrl, hash);
 
-        var expectedPackageURL = new PackageURL(
+        var expectedPackageURL = new PackageUrl(
             type: "swift",
             @namespace: "github.com/Alamofire",
             name: name,
@@ -124,7 +124,7 @@ public void PackageURL_ShouldReturnCorrectPackageURL_GithubHostname_Alternate()
 
         var component = new SwiftComponent(name, version, packageUrl, hash);
 
-        var expectedPackageURL = new PackageURL(
+        var expectedPackageURL = new PackageUrl(
             type: "swift",
             @namespace: "github.com/Alamofire",
             name: name,
@@ -149,7 +149,7 @@ public void PackageURL_ShouldReturnCorrectPackageURL_OtherHostname()
 
         var component = new SwiftComponent(name, version, packageUrl, hash);
 
-        var expectedPackageURL = new PackageURL(
+        var expectedPackageURL = new PackageUrl(
             type: "swift",
             @namespace: "otherhostname.com",
             name: name,

Original file line number	Diff line number	Diff line change
`@@ -104,7 +104,7 @@ public class YourEcosystemComponent : TypedComponent`
`104`	`104`	`public string Version { get; set; }`
`105`	`105`
`106`	`106`	`public override ComponentType Type => ComponentType.YourType;`
`107`		`- public override PackageURL PackageUrl => new PackageURL("your-type", null, this.Name, this.Version, null, null);`
	`107`	`+ public override PackageUrl PackageUrl => new PackageUrl("your-type", null, this.Name, this.Version, null, null);`
`108`	`108`	`protected override string ComputeId() => $"{this.Name} {this.Version} - {this.Type}";`
`109`	`109`	`}`
`110`	`110`	```
Original file line number	Diff line number	Diff line change
`@@ -547,4 +547,4 @@`
`547`	`547`	`"resultCode",`
`548`	`548`	`"sourceDirectory"`
`549`	`549`	`]`
`550`		`-}`
	`550`	`+}`
Original file line number	Diff line number	Diff line change
`@@ -44,7 +44,7 @@ public CargoComponent(string name, string version, string author = null, string`
`44`	`44`	`public override ComponentType Type => ComponentType.Cargo;`
`45`	`45`
`46`	`46`	`[JsonPropertyName("packageUrl")]`
`47`		`- public override PackageURL PackageUrl => new PackageURL("cargo", string.Empty, this.Name, this.Version, null, string.Empty);`
	`47`	`+ public override PackageUrl PackageUrl => new PackageUrl("cargo", string.Empty, this.Name, this.Version, null, string.Empty);`
`48`	`48`
`49`	`49`	`protected override string ComputeBaseId() => $"{this.Name} {this.Version} - {this.Type}";`
`50`	`50`	`}`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public ConanComponent(string name, string version, string previous, string packa`
`38`	`38`	`public override ComponentType Type => ComponentType.Conan;`
`39`	`39`
`40`	`40`	`[JsonPropertyName("packageUrl")]`
`41`		`- public override PackageURL PackageUrl => new PackageURL("conan", string.Empty, this.Name, this.Version, null, string.Empty);`
	`41`	`+ public override PackageUrl PackageUrl => new PackageUrl("conan", string.Empty, this.Name, this.Version, null, string.Empty);`
`42`	`42`
`43`	`43`	`protected override string ComputeBaseId() => $"{this.Name} {this.Version} - {this.Type}";`
`44`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -39,15 +39,15 @@ public CppSdkComponent(string name, string version)`
`39`	`39`	`public override ComponentType Type => ComponentType.CppSdk;`
`40`	`40`
`41`	`41`	`[JsonPropertyName("packageUrl")]`
`42`		`- public override PackageURL PackageUrl`
	`42`	`+ public override PackageUrl PackageUrl`
`43`	`43`	`{`
`44`	`44`	`get`
`45`	`45`	`{`
`46`	`46`	`var qualifiers = new SortedDictionary<string, string>`
`47`	`47`	`{`
`48`	`48`	`{ "type", "cppsdk" },`
`49`	`49`	`};`
`50`		`- return new PackageURL("generic", null, this.Name, this.Version, qualifiers, null);`
	`50`	`+ return new PackageUrl("generic", null, this.Name, this.Version, qualifiers, null);`
`51`	`51`	`}`
`52`	`52`	`}`
`53`	`53`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ public LinuxComponent(string distribution, string release, string name, string v`
`46`	`46`	`public override ComponentType Type => ComponentType.Linux;`
`47`	`47`
`48`	`48`	`[JsonPropertyName("packageUrl")]`
`49`		`- public override PackageURL PackageUrl`
	`49`	`+ public override PackageUrl PackageUrl`
`50`	`50`	`{`
`51`	`51`	`get`
`52`	`52`	`{`
`@@ -63,7 +63,7 @@ public override PackageURL PackageUrl`
`63`	`63`
`64`	`64`	`if (packageType != null)`
`65`	`65`	`{`
`66`		`- return new PackageURL(packageType, this.Distribution, this.Name, this.Version, null, null);`
	`66`	`+ return new PackageUrl(packageType, this.Distribution, this.Name, this.Version, null, null);`
`67`	`67`	`}`
`68`	`68`
`69`	`69`	`return null;`
Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ public MavenComponent()`
`31`	`31`	`public override ComponentType Type => ComponentType.Maven;`
`32`	`32`
`33`	`33`	`[JsonPropertyName("packageUrl")]`
`34`		`- public override PackageURL PackageUrl => new PackageURL("maven", this.GroupId, this.ArtifactId, this.Version, null, null);`
	`34`	`+ public override PackageUrl PackageUrl => new PackageUrl("maven", this.GroupId, this.ArtifactId, this.Version, null, null);`
`35`	`35`
`36`	`36`	`protected override string ComputeBaseId() => $"{this.GroupId} {this.ArtifactId} {this.Version} - {this.Type}";`
`37`	`37`	`}`
Original file line number	Diff line number	Diff line change
`@@ -36,7 +36,7 @@ public NpmComponent(string name, string version, string hash = null, NpmAuthor a`
`36`	`36`	`public override ComponentType Type => ComponentType.Npm;`
`37`	`37`
`38`	`38`	`[JsonPropertyName("packageUrl")]`
`39`		`- public override PackageURL PackageUrl => new PackageURL("npm", null, this.Name, this.Version, null, null);`
	`39`	`+ public override PackageUrl PackageUrl => new PackageUrl("npm", null, this.Name, this.Version, null, null);`
`40`	`40`
`41`	`41`	`protected override string ComputeBaseId() => $"{this.Name} {this.Version} - {this.Type}";`
`42`	`42`	`}`