Merge pull request github#16796 from owen-mc/go/fix/package-vendor

owen-mc · web-flow · commit 513ec16691af · 2024-06-22T07:54:18.000+01:00
Go: Fix bug removing "vendor/" from package paths
diff --git a/go/ql/lib/semmle/go/Packages.qll b/go/ql/lib/semmle/go/Packages.qll
@@ -14,7 +14,7 @@ class Package extends @package {
   /** Gets the path of this package. */
   string getPath() {
     exists(string fullPath | packages(this, _, fullPath, _) |
-      result = fullPath.regexpReplaceAll("^.*/vendor/", "")
+      result = fullPath.regexpReplaceAll("^.*\\bvendor/", "")
     )
   }
 

Original file line number	Diff line number	Diff line change
`@@ -14,7 +14,7 @@ class Package extends @package {`
`14`	`14`	`/** Gets the path of this package. */`
`15`	`15`	`string getPath() {`
`16`	`16`	`exists(string fullPath \| packages(this, _, fullPath, _) \|`
`17`		`- result = fullPath.regexpReplaceAll("^.*/vendor/", "")`
	`17`	`+ result = fullPath.regexpReplaceAll("^.*\\bvendor/", "")`
`18`	`18`	`)`
`19`	`19`	`}`
`20`	`20`