Skip to content

Commit 3926021

Browse files
sindrerh2tommytroenybelMekk
committed
refactor: fix workloadtype mapping
Co-authored-by: tommytroen <[email protected]> Co-authored-by: ybelmekk <[email protected]>
1 parent 313b081 commit 3926021

File tree

2 files changed

+26
-3
lines changed

2 files changed

+26
-3
lines changed

internal/api/grpcvulnerabilities/vulnerabilities.go

+3-3
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ func (s *Server) ListVulnerabilities(ctx context.Context, request *vulnerabiliti
2929
v, err := s.querier.ListVulnerabilities(ctx, sql.ListVulnerabilitiesParams{
3030
Cluster: request.GetFilter().Cluster,
3131
Namespace: request.GetFilter().Namespace,
32-
WorkloadType: request.GetFilter().WorkloadType,
32+
WorkloadType: request.GetFilter().FuzzyWorkloadType(),
3333
WorkloadName: request.GetFilter().Workload,
3434
ImageName: request.GetFilter().ImageName,
3535
ImageTag: request.GetFilter().ImageTag,
@@ -77,7 +77,7 @@ func (s *Server) ListVulnerabilities(ctx context.Context, request *vulnerabiliti
7777
total, err := s.querier.CountVulnerabilities(ctx, sql.CountVulnerabilitiesParams{
7878
Cluster: request.GetFilter().Cluster,
7979
Namespace: request.GetFilter().Namespace,
80-
WorkloadType: request.GetFilter().WorkloadType,
80+
WorkloadType: request.GetFilter().FuzzyWorkloadType(),
8181
WorkloadName: request.GetFilter().Workload,
8282
IncludeSuppressed: request.IncludeSuppressed,
8383
})
@@ -186,7 +186,7 @@ func (s *Server) ListSuppressedVulnerabilities(ctx context.Context, request *vul
186186
total, err := s.querier.CountSuppressedVulnerabilities(ctx, sql.CountSuppressedVulnerabilitiesParams{
187187
Cluster: filter.Cluster,
188188
Namespace: filter.Namespace,
189-
WorkloadType: filter.WorkloadType,
189+
WorkloadType: filter.FuzzyWorkloadType(),
190190
WorkloadName: filter.Workload,
191191
ImageName: filter.ImageName,
192192
ImageTag: filter.ImageTag,

pkg/api/vulnerabilities/options.go

+23
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,13 @@ type Option interface {
1212

1313
type OrderByField string
1414

15+
type WorkLoadType string
16+
17+
const (
18+
WorkloadTypeApp WorkLoadType = "app"
19+
WorkloadTypeJob WorkLoadType = "job"
20+
)
21+
1522
const (
1623
OrderBySeverity OrderByField = "severity"
1724
OrderByPackage OrderByField = "package"
@@ -179,3 +186,19 @@ func applyOptions(opts ...Option) *options {
179186
}
180187
return o
181188
}
189+
190+
func (f *Filter) FuzzyWorkloadType() *string {
191+
if f.WorkloadType == nil {
192+
return nil
193+
}
194+
app := string(WorkloadTypeApp)
195+
job := string(WorkloadTypeJob)
196+
197+
switch *f.WorkloadType {
198+
case "app", "APP", "application", "Application", "APPLICATION":
199+
return &app
200+
case "job", "Job", "JOB", "Naisjob", "NAISJOB", "NaisJob":
201+
return &job
202+
}
203+
return f.WorkloadType
204+
}

0 commit comments

Comments
 (0)