Skip to content

Whitelist /system/probes/start like /system/probes/health - #511

Merged
SachinMali merged 2 commits into
adobe:mainfrom
moalfamaria:SKYOPS-152446-whitelist-probes-start
Sep 29, 2026
Merged

SachinMali merged 2 commits into
adobe:mainfrom
moalfamaria:SKYOPS-152446-whitelist-probes-start

Conversation

@moalfamaria

Copy link
Copy Markdown

Summary

Mirrors Granite/dispatcher-k8s-base#644: the runtime dispatcher_vhost.conf now also bypasses Basic Auth / dispatcher filters on /system/probes/start (the Kubernetes startup probe), the same way it already does for /system/probes/health. This keeps the illustrative copy shipped in this repo (dispatcher/src/conf.d/dispatcher_vhost.conf) in sync with the actual runtime file.

Changes

  • Added a <Location "/system/probes/start"> block mirroring the existing /system/probes/health block, in both the "managed rewrite maps not-configured" and "configured + ready" branches.

JIRA: https://jira.corp.adobe.com/browse/SKYOPS-152446

@moalfamaria
moalfamaria force-pushed the SKYOPS-152446-whitelist-probes-start branch 2 times, most recently from e83ead7 to ef38e1d Compare August 5, 2026 11:16
Mirrors Granite/dispatcher-k8s-base#644: the runtime dispatcher_vhost.conf
now also bypasses Basic Auth / dispatcher filters on /system/probes/start
(K8s startup probe), the same way it already does for /system/probes/health.
This keeps the illustrative copy shipped here in sync.

JIRA: https://jira.corp.adobe.com/browse/SKYOPS-152446
@moalfamaria
moalfamaria force-pushed the SKYOPS-152446-whitelist-probes-start branch from ef38e1d to 5e8ca1d Compare August 5, 2026 11:20

@cisbiceanu cisbiceanu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

…d standardized whitespace formatting in dispatcher_vhost.conf.
@SachinMali
SachinMali merged commit 5355b51 into adobe:main Sep 29, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants