Skip to content

STAC-24889: authenticate Cerberus calls - #453

Merged
VioletCranberry merged 1 commit into
STAC-25142-agent-lint-unitfrom
STAC-24889-cerberus-auth
Aug 5, 2026
Merged

STAC-24889: authenticate Cerberus calls#453
VioletCranberry merged 1 commit into
STAC-25142-agent-lint-unitfrom
STAC-24889-cerberus-auth

Conversation

@VioletCranberry

Copy link
Copy Markdown

Cerberus now requires CI callers to authenticate with a shared bearer token
(StackVista/cerberus#4) - the endpoint was previously reachable by anyone who knew the URL.

.cerberus/cerberus_notify_failure.sh now:

  • sends Authorization: Bearer ${CERBERUS_API_TOKEN}
  • suspends xtrace around the curl and drops --verbose - the script runs under
    set -exuo pipefail, so either would echo the token into the job log
  • uses the ${VAR?message} form so a missing token fails loudly instead of silently
    sending Bearer

Targets stackstate-7.78.2 because that is the default branch and .gitlab-ci.yml pins the
notify-on-master-fail job to only: stackstate-7.78.2 - that is where the live caller is.

The script sends action: notify only, so it touches no branch protection.
CERBERUS_API_TOKEN is already provisioned for this repo.

Safe to merge before the Cerberus Lambda is redeployed: the current one ignores the header.

Relates to STAC-24889

Cerberus now requires CI callers to present a shared bearer token. Send it from
the notify script.

Suspend xtrace around the curl and drop --verbose: the script runs under
set -exuo pipefail, so either would echo the token into the job log.
@VioletCranberry
VioletCranberry changed the base branch from stackstate-7.78.2 to STAC-25142-agent-lint-unit August 5, 2026 13:04
@VioletCranberry
VioletCranberry merged commit 5ea461d into STAC-25142-agent-lint-unit Aug 5, 2026
@VioletCranberry
VioletCranberry deleted the STAC-24889-cerberus-auth branch August 5, 2026 13:06
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.

2 participants