Merge pull request #1008 from danielhardej/danielhardej-patch-20251023

dangoor · web-flow · commit ebabd31cea65 · 2025-11-07T18:20:38.000-05:00
Fix PURL parsing to prevent mismatch for scoped packages
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -43,7 +43,7 @@
     "zod": "^3.24.1"
   },
   "devDependencies": {
-    "@types/jest": "^29.5.12",
+    "@types/jest": "^29.5.14",
     "@types/node": "^20",
     "@types/spdx-expression-parse": "^3.0.4",
     "@typescript-eslint/eslint-plugin": "^6.21.0",
diff --git a/src/licenses.ts b/src/licenses.ts
@@ -174,7 +174,7 @@ async function groupChanges(
         return true
       }
 
-      const changeAsPackageURL = parsePURL(encodeURI(change.package_url))
+      const changeAsPackageURL = parsePURL(change.package_url)
 
       // We want to find if the licenseExclusion list contains the PackageURL of the Change
       // If it does, we want to filter it out and therefore return false

Original file line number	Diff line number	Diff line change
`@@ -174,7 +174,7 @@ async function groupChanges(`
`174`	`174`	`return true`
`175`	`175`	`}`
`176`	`176`
`177`		`- const changeAsPackageURL = parsePURL(encodeURI(change.package_url))`
	`177`	`+ const changeAsPackageURL = parsePURL(change.package_url)`
`178`	`178`
`179`	`179`	`// We want to find if the licenseExclusion list contains the PackageURL of the Change`
`180`	`180`	`// If it does, we want to filter it out and therefore return false`