Skip to content

Commit

Permalink
security issues due to fixed ns in crb (#14)
Browse files Browse the repository at this point in the history
testd with
OpenShift Server Version: 4.12.0
Kubernetes Version: v1.25.4
  • Loading branch information
rbaumgar authored Jan 29, 2023
1 parent 569e559 commit 06115e7
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions deploy/dynu-webhook/templates/rbac.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ subjects:
- apiGroup: ""
kind: ServiceAccount
name: {{ .Values.certManager.serviceAccountName }}
namespace: {{ .Values.certManager.namespace }}
namespace: {{ .Release.Namespace }}

---
# Grant cert-manager permission to read secrets from cert-manager ns (i.e. to read the api key for dynu)
Expand Down Expand Up @@ -140,11 +140,6 @@ subjects:
kind: ServiceAccount
name: {{ include "dynu-webhook.fullname" . }}
namespace: {{ .Release.Namespace }}
- apiGroup: ""
kind: ServiceAccount
name: {{ .Values.certManager.serviceAccountName }}
namespace: {{ .Values.certManager.namespace }}


---

Expand Down

0 comments on commit 06115e7

Please sign in to comment.