python syntax fix + remove port override in mutatewebhook

sse-secure-systems · Feb 4, 2022 · 849472c · 849472c
1 parent a5a3bc4
commit 849472c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/connaisseur/flask_application.py b/connaisseur/flask_application.py
@@ -51,8 +51,14 @@ def handle_alert_config_error(err):
 
 
 @APP.route("/mutate", methods=["POST"])
-@metrics.counter('mutate_requests_total', 'Total number of mutate requests',labels={'allowed': lambda r: r.get_json(silent=True)['response']['allowed'],
-    'status_code': lambda r: r.get_json(silent=True)['response']['status']['code']})
+@metrics.counter(
+    "mutate_requests_total",
+    "Total number of mutate requests",
+    labels={
+        "allowed": lambda r: r.get_json(silent=True)["response"]["allowed"],
+        "status_code": lambda r: r.get_json(silent=True)["response"]["status"]["code"],
+    },
+)
 def mutate():
     """
     Handle the '/mutate' path and accept CREATE and UPDATE requests.

diff --git a/helm/templates/certificate_webhook-conf.yaml b/helm/templates/certificate_webhook-conf.yaml
@@ -71,7 +71,6 @@ webhooks:
         name: {{ .Chart.Name }}-svc
         namespace: {{ .Release.Namespace }}
         path: /mutate
-        port: {{ .Values.service.port }}
       caBundle: {{ default ($certs.Cert | b64enc) (include "getInstalledTLSCert" .) }}
     rules:
       - operations: ["CREATE", "UPDATE"]