Skip to content
This repository has been archived by the owner on Jan 23, 2025. It is now read-only.

bug fix : misspelling #1311

Merged
merged 4 commits into from
May 25, 2023
Merged

bug fix : misspelling #1311

merged 4 commits into from
May 25, 2023

Conversation

testwill
Copy link
Contributor

@testwill testwill commented May 7, 2023

No description provided.

@testwill testwill requested review from giorod3 and simar7 as code owners May 7, 2023 10:54
@simar7
Copy link
Member

simar7 commented May 17, 2023

could you update here as well

defsec/rules/kubernetes/lib/kubernetes.rego

Lines 143 to 150 in 673ff3a

priviledge_escalation_allowed(c) {
not has_field(c, "securityContext")
}
priviledge_escalation_allowed(c) {
has_field(c, "securityContext")
has_field(c.securityContext, "allowPrivilegeEscalation")
}

simar7
simar7 approved these changes May 25, 2023
View reviewed changes
Copy link
Member

@simar7 simar7 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thanks!

@simar7 simar7 merged commit 02044c0 into aquasecurity:master May 25, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@simar7 simar7 simar7 approved these changes

@giorod3 giorod3 Awaiting requested review from giorod3

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@testwill @simar7