Skip to content

Commit de50bd9

Browse files
committed
feat: rename property name
1 parent 573a6f9 commit de50bd9

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

eval/cmdeval.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ func (cv commandEvaluate) evalPolicy(commands []string, cmdExec cmd, evalExpr st
8080
policyEvalResults = append(policyEvalResults, policyResult)
8181
}
8282
for _, per := range policyEvalResults {
83-
if returnVal, ok := per.ReturnValues["allow"]; ok {
83+
if returnVal, ok := per.ReturnValues["match"]; ok {
8484
val, err := strconv.ParseBool(returnVal)
8585
if err != nil {
8686
continue

eval/cmdeval_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,10 +57,10 @@ func TestEvalPolicy(t *testing.T) {
5757
policy string
5858
want bool
5959
}{
60-
{name: "two command and deny policy match", evalExpr: "'${0}' != '';&& [${1} MATCH no_permission.policy QUERY itsio.policy_eval RETURN allow,name]", cmd: []string{"kubectl get pods --no-headers -o custom-columns=\":metadata.name\"",
60+
{name: "two command and deny policy match", evalExpr: "'${0}' != '';&& [${1} MATCH no_permission.policy QUERY itsio.policy_eval RETURN match,name]", cmd: []string{"kubectl get pods --no-headers -o custom-columns=\":metadata.name\"",
6161
"kubectl get pod ${0} -o json"},
6262
policy: AllowPolicy, want: true},
63-
{name: "two command and deny policy expr not match", evalExpr: "'${0}' == '';&& [${1} MATCH no_permission.policy QUERY itsio.policy_eval RETURN allow,name]", cmd: []string{"kubectl get pods --no-headers -o custom-columns=\":metadata.name\"",
63+
{name: "two command and deny policy expr not match", evalExpr: "'${0}' == '';&& [${1} MATCH no_permission.policy QUERY itsio.policy_eval RETURN match,name]", cmd: []string{"kubectl get pods --no-headers -o custom-columns=\":metadata.name\"",
6464
"kubectl get pod ${0} -o json"},
6565
policy: NotAllowPolicy, want: false},
6666
}

0 commit comments

Comments
 (0)