Bug
In src/server/deployers/otel.ts line 96, the condition:
if (config.mode === "kubernetes" || config.mode === "openshift" && config.otelExperimentId)
evaluates as kubernetes || (openshift && experimentId) because && binds tighter than ||. This means the bearertokenauth extension is always registered on Kubernetes regardless of whether an experiment ID is set — but on OpenShift it's only registered when one is.
Expected behavior
bearertokenauth should only be registered when both:
- The mode is
kubernetes or openshift, and
- An
otelExperimentId is configured
Fix
Wrap the mode check in parentheses:
if ((config.mode === "kubernetes" || config.mode === "openshift") && config.otelExperimentId)
Bug
In
src/server/deployers/otel.tsline 96, the condition:evaluates as
kubernetes || (openshift && experimentId)because&&binds tighter than||. This means thebearertokenauthextension is always registered on Kubernetes regardless of whether an experiment ID is set — but on OpenShift it's only registered when one is.Expected behavior
bearertokenauthshould only be registered when both:kubernetesoropenshift, andotelExperimentIdis configuredFix
Wrap the mode check in parentheses: