Skip to content
pr

Add capability to set response headers in OPA filter #3195

Sign in to view logs

Add capability to set response headers in OPA filter

Add capability to set response headers in OPA filter #3195

Triggered via pull request August 15, 2023 16:15
@PushpalankaPushpalanka
synchronize #2517
Pushpalanka:opa-response
Status Failure
Total duration 9m 26s
Artifacts –

pr.yaml

on: pull_request
semgrep
15s
semgrep
check-race
9m 13s
check-race
tests
4m 35s
tests
Fit to window
Zoom out
Zoom in

Annotations

5 errors
tests: filters/openpolicyagent/opaauthorizerequest/opaauthorizerequest.go#L137
should omit nil check; len() for []*github.com/envoyproxy/go-control-plane/envoy/config/core/v3.HeaderValueOption is defined as zero (S1009)
tests: filters/openpolicyagent/opaauthorizerequest/opaauthorizerequest.go#L169
headersToAdd refers to the result of a failed type assertion and is a zero value, not the value that was being type-asserted (SA9008)
tests: filters/openpolicyagent/opaauthorizerequest/opaauthorizerequest.go#L166
this is the variable being read
tests: filters/openpolicyagent/opaauthorizerequest/opaauthorizerequest.go#L166
this is the variable being shadowed
tests
Process completed with exit code 2.