mirror of
https://github.com/anchore/syft.git
synced 2025-11-17 16:33:21 +01:00
Fix package url for Go modules with no / (#1092)
This commit is contained in:
parent
b3a7b912e1
commit
2f1aa33ce8
@ -47,8 +47,10 @@ func URL(p Package, release *linux.Release) string {
|
|||||||
case p.Type == GoModulePkg:
|
case p.Type == GoModulePkg:
|
||||||
re := regexp.MustCompile(`(/)[^/]*$`)
|
re := regexp.MustCompile(`(/)[^/]*$`)
|
||||||
fields := re.Split(p.Name, -1)
|
fields := re.Split(p.Name, -1)
|
||||||
namespace = fields[0]
|
if len(fields) > 1 {
|
||||||
name = strings.TrimPrefix(p.Name, namespace+"/")
|
namespace = fields[0]
|
||||||
|
name = strings.TrimPrefix(p.Name, namespace+"/")
|
||||||
|
}
|
||||||
case p.Type == NpmPkg:
|
case p.Type == NpmPkg:
|
||||||
fields := strings.SplitN(p.Name, "/", 2)
|
fields := strings.SplitN(p.Name, "/", 2)
|
||||||
if len(fields) > 1 {
|
if len(fields) > 1 {
|
||||||
|
|||||||
@ -25,6 +25,15 @@ func TestPackageURL(t *testing.T) {
|
|||||||
},
|
},
|
||||||
expected: "pkg:golang/github.com/anchore/syft@v0.1.0",
|
expected: "pkg:golang/github.com/anchore/syft@v0.1.0",
|
||||||
},
|
},
|
||||||
|
{
|
||||||
|
name: "golang short name",
|
||||||
|
pkg: Package{
|
||||||
|
Name: "go.opencensus.io",
|
||||||
|
Version: "v0.23.0",
|
||||||
|
Type: GoModulePkg,
|
||||||
|
},
|
||||||
|
expected: "pkg:golang/go.opencensus.io@v0.23.0",
|
||||||
|
},
|
||||||
{
|
{
|
||||||
name: "pub",
|
name: "pub",
|
||||||
pkg: Package{
|
pkg: Package{
|
||||||
@ -237,7 +246,7 @@ func TestPackageURL(t *testing.T) {
|
|||||||
|
|
||||||
for _, test := range tests {
|
for _, test := range tests {
|
||||||
t.Run(test.name, func(t *testing.T) {
|
t.Run(test.name, func(t *testing.T) {
|
||||||
if test.pkg.Type != "" {
|
if test.pkg.Type != "" && !contains(pkgTypes, string(test.pkg.Type)) {
|
||||||
pkgTypes = append(pkgTypes, string(test.pkg.Type))
|
pkgTypes = append(pkgTypes, string(test.pkg.Type))
|
||||||
}
|
}
|
||||||
actual := URL(test.pkg, test.distro)
|
actual := URL(test.pkg, test.distro)
|
||||||
@ -250,3 +259,13 @@ func TestPackageURL(t *testing.T) {
|
|||||||
}
|
}
|
||||||
assert.ElementsMatch(t, expectedTypes.List(), pkgTypes, "missing one or more package types to test against (maybe a package type was added?)")
|
assert.ElementsMatch(t, expectedTypes.List(), pkgTypes, "missing one or more package types to test against (maybe a package type was added?)")
|
||||||
}
|
}
|
||||||
|
|
||||||
|
func contains(values []string, val string) bool {
|
||||||
|
for _, v := range values {
|
||||||
|
if val == v {
|
||||||
|
return true
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return false
|
||||||
|
}
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user