Skip to content

Commit 7175af7

Browse files
Merge pull request #1274 from Checkmarx/other/Remove-filepath-from-ossRealTime-ignore-flow
Remove filepath from ignore logic in oss real time (AST-109999)
2 parents 0eaa485 + ec68432 commit 7175af7

File tree

5 files changed

+10
-13
lines changed

5 files changed

+10
-13
lines changed

go.mod

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ require (
3838
github.com/santhosh-tekuri/jsonschema/v6 v6.0.2 // indirect
3939
go.yaml.in/yaml/v2 v2.4.2 // indirect
4040
go.yaml.in/yaml/v3 v3.0.3 // indirect
41-
dario.cat/mergo v1.0.1 // indirect
41+
dario.cat/mergo v1.0.1 // indirect
4242
github.com/AdaLogics/go-fuzz-headers v0.0.0-20240806141605-e8a1dd7889d6 // indirect
4343
github.com/AdamKorcz/go-118-fuzz-build v0.0.0-20240914100643-eb91380d8434 // indirect
4444
github.com/Azure/go-ansiterm v0.0.0-20250102033503-faa5f7b0171c // indirect

internal/commands/data/checkmarxIgnoredTempList.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
{
33
"PackageManager": "npm",
44
"PackageName": "coa",
5-
"PackageVersion": "3.1.3",
6-
"FilePath": "../../../commands/data/manifests/package.json"
5+
"PackageVersion": "3.1.3"
6+
77
}
88
]

internal/commands/data/checkmarxIgnoredTempListCsproj.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22
{
33
"PackageManager": "nuget",
44
"PackageName": "Microsoft.Extensions.Caching.Memory",
5-
"PackageVersion": "6.0.3",
6-
"FilePath": "../../../commands/data/manifests/test.csproj"
5+
"PackageVersion": "6.0.3"
6+
77

88
}
99
]

internal/services/realtimeengine/ossrealtime/config.go

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,23 +22,22 @@ type OssPackageResults struct {
2222
Packages []OssPackage `json:"Packages"`
2323
}
2424

25-
func composeID(packageManager, packageName, packageVersion, filePath string) string {
26-
return fmt.Sprintf("%s_%s_%s_%s", packageManager, packageName, packageVersion, filePath)
25+
func composeID(packageManager, packageName, packageVersion string) string {
26+
return fmt.Sprintf("%s_%s_%s", packageManager, packageName, packageVersion)
2727
}
2828

2929
func (p *OssPackage) GetID() string {
30-
return composeID(p.PackageManager, p.PackageName, p.PackageVersion, p.FilePath)
30+
return composeID(p.PackageManager, p.PackageName, p.PackageVersion)
3131
}
3232

3333
type IgnoredPackage struct {
3434
PackageManager string `json:"PackageManager"`
3535
PackageName string `json:"PackageName"`
3636
PackageVersion string `json:"PackageVersion"`
37-
FilePath string `json:"FilePath"`
3837
}
3938

4039
func (p IgnoredPackage) GetID() string {
41-
return composeID(p.PackageManager, p.PackageName, p.PackageVersion, p.FilePath)
40+
return composeID(p.PackageManager, p.PackageName, p.PackageVersion)
4241
}
4342

4443
type Vulnerability struct {

internal/services/realtimeengine/ossrealtime/oss-realtime.go

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -95,9 +95,7 @@ func (o *OssRealtimeService) RunOssRealtimeScan(filePath, ignoredFilePath string
9595
ignoreMap := buildIgnoreMap(ignoredPkgs)
9696
response.Packages = filterIgnoredPackages(response.Packages, ignoreMap)
9797
}
98-
for i := range response.Packages {
99-
response.Packages[i].FilePath = filePath
100-
}
98+
10199
return response, nil
102100
}
103101

0 commit comments

Comments
 (0)