From 8e532b903b84dbe20973fb72620e44e3e5af34aa Mon Sep 17 00:00:00 2001
From: Julian <julian@jpinzer.me>
Date: Wed, 7 Dec 2022 21:09:50 +0000
Subject: [PATCH 1/2] Fix issues handling golang packages with multi-part
 namespaces. Added a test for it as well.

---
 src/PackageUrl.cs                     |  8 +++++++-
 tests/TestAssets/test-suite-data.json | 12 ++++++++++++
 2 files changed, 19 insertions(+), 1 deletion(-)

diff --git a/src/PackageUrl.cs b/src/PackageUrl.cs
index ae49a2e..175e32e 100644
--- a/src/PackageUrl.cs
+++ b/src/PackageUrl.cs
@@ -42,6 +42,11 @@ namespace PackageUrl
     [Serializable]
     public sealed class PackageURL
     {
+        /// <summary>
+        /// The url encoding of /.
+        /// </summary>
+        private const string EncodedSlash = "%2F";
+
         private static readonly Regex s_typePattern = new Regex("^[a-zA-Z][a-zA-Z0-9.+-]+$", RegexOptions.Compiled);
 
         /// <summary>
@@ -136,6 +141,7 @@ public override string ToString()
             if (Namespace != null)
             {
                 purl.Append(WebUtility.UrlEncode(Namespace));
+                purl.Replace(EncodedSlash, "/");
                 purl.Append('/');
             }
             if (Name != null)
@@ -238,7 +244,7 @@ private void Parse(string purl)
                 int i;
                 for (i = 1; i < firstPartArray.Length - 2; ++i)
                 {
-                    @namespace += firstPartArray[i] + ',';
+                    @namespace += firstPartArray[i] + '/';
                 }
                 @namespace += firstPartArray[i];
 
diff --git a/tests/TestAssets/test-suite-data.json b/tests/TestAssets/test-suite-data.json
index 6f97708..25fdf13 100644
--- a/tests/TestAssets/test-suite-data.json
+++ b/tests/TestAssets/test-suite-data.json
@@ -47,6 +47,18 @@
     "subpath": "googleapis/api/annotations",
     "is_invalid": false
   },
+  {
+    "description": "valid go purl with version, subpath, and multi-part namespace",
+    "purl": "pkg:GOLANG/github.com/gorilla/context@234fd47e07d1004f0aed9c#api/",
+    "canonical_purl": "pkg:golang/github.com/gorilla/context@234fd47e07d1004f0aed9c#api",
+    "type": "golang",
+    "namespace": "github.com/gorilla",
+    "name": "context",
+    "version": "234fd47e07d1004f0aed9c",
+    "qualifiers": null,
+    "subpath": "api",
+    "is_invalid": false
+  },
   {
     "description": "bitbucket namespace and name should be lowercased",
     "purl": "pkg:bitbucket/birKenfeld/pyGments-main@244fd47e07d1014f0aed9c",

From 569fb16ccaa4b9f970a87bec159a635fbf09ab30 Mon Sep 17 00:00:00 2001
From: Julian <julian@jpinzer.me>
Date: Thu, 8 Dec 2022 12:37:48 -0500
Subject: [PATCH 2/2] Update src/PackageUrl.cs

Match packageurl-js implementation for namespace encoding.

Co-authored-by: Adeel Mujahid <3840695+am11@users.noreply.github.com>
---
 src/PackageUrl.cs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/PackageUrl.cs b/src/PackageUrl.cs
index 175e32e..324ed2c 100644
--- a/src/PackageUrl.cs
+++ b/src/PackageUrl.cs
@@ -140,8 +140,8 @@ public override string ToString()
             purl.Append('/');
             if (Namespace != null)
             {
-                purl.Append(WebUtility.UrlEncode(Namespace));
-                purl.Replace(EncodedSlash, "/");
+                string encodedNamespace = WebUtility.UrlEncode(Namespace).Replace(EncodedSlash, "/");
+                purl.Append(encodedNamespace);
                 purl.Append('/');
             }
             if (Name != null)