Skip to content

Commit 39225ab

Browse files
authored
Merge pull request #119 from emsearcy/fix/lfxv2-1336-heimdall-403-error-message
fix: explicitly set Heimdall verbose: false in serve.respond
2 parents afd6d12 + 90b62ba commit 39225ab

1 file changed

Lines changed: 7 additions & 3 deletions

File tree

charts/lfx-platform/values.yaml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ fga-operator:
6969
# Non-chart value
7070
store: "lfx-core"
7171
controllerManager:
72-
openFgaUrlEnvVar: 'http://lfx-platform-openfga:8080'
72+
openFgaUrlEnvVar: "http://lfx-platform-openfga:8080"
7373
# This value needs to be set in order for the operator to start, but
7474
# because no auth is required for the openfga API this value will be
7575
# ignored by openfga
@@ -175,7 +175,6 @@ heimdall:
175175
image:
176176
tag: 0.17.6
177177

178-
179178
deployment:
180179
replicaCount: 1
181180
resources:
@@ -222,6 +221,11 @@ heimdall:
222221
- "10.0.0.0/8"
223222
- "172.16.0.0/12"
224223
- "192.168.0.0/16"
224+
respond:
225+
# Keep verbose: false (the Heimdall default) so that error responses
226+
# return a bare HTTP status code with no body, rather than leaking
227+
# internal pipeline details (e.g., "expression 1 failed") to clients.
228+
verbose: false
225229

226230
mechanisms:
227231
authenticators:
@@ -250,7 +254,7 @@ heimdall:
250254
# the client_id. Client IDs can collide with usernames (and GJSON
251255
# doesn't let us do array concatenation to add a literal prefix), so
252256
# the `sub` claim should NOT be used downstream.
253-
id: '[username,client_id].0'
257+
id: "[username,client_id].0"
254258
contextualizers:
255259
- id: oidc_contextualizer
256260
type: generic

0 commit comments

Comments
 (0)