From ae3635daa635059eb5e062f8bc9c808e7e8d379e Mon Sep 17 00:00:00 2001 From: Ryan Gonzalez <ryan.gonzalez@collabora.com> Date: Thu, 6 Jun 2024 15:12:03 -0500 Subject: [PATCH] chart: Remove unused Traefik middleware Apparently, while copying the contents of the old aptly-repository over to here, I accidentally included this middleware file that was sticking around in my local copy from back when we were deciding how exactly to set up authentication. The original idea was that we'd use oathkeeper as a decision maker only, rather than a proxy as it is now, but this was changed in order to avoid dealing with CRDs. Of course, this file got stuck in here, which spoiled that goal just a tiny bit. In fact, I *literally reviewed changes to it* in !13 that were explicitly tied to the exact CRD issues we wanted to avoid in the first place. https://phabricator.apertis.org/T10135 Signed-off-by: Ryan Gonzalez <ryan.gonzalez@collabora.com> --- chart/templates/middleware.yaml | 13 ------------- 1 file changed, 13 deletions(-) delete mode 100644 chart/templates/middleware.yaml diff --git a/chart/templates/middleware.yaml b/chart/templates/middleware.yaml deleted file mode 100644 index ccebb7f3..00000000 --- a/chart/templates/middleware.yaml +++ /dev/null @@ -1,13 +0,0 @@ -apiVersion: traefik.containo.us/v1alpha1 -kind: Middleware -metadata: - name: {{ include "aptly.name" . }}-auth - labels: - {{- include "aptly.labels" . | nindent 4 }} -spec: - forwardAuth: - address: http://oathkeeper-api:4456/decisions - authResponseHeaders: - - Authorization - # basicAuth: - # secret: {{ include "aptly.name" . }}-auth -- GitLab