Skip to content

codegen-orchestrator-1707: Propagate and redact the GitHub webhook secret - #847

Merged
vladmesh merged 3 commits into
mainfrom
pipeline/codegen-orchestrator-1707
Oct 18, 2026
Merged

vladmesh merged 3 commits into
mainfrom
pipeline/codegen-orchestrator-1707

Conversation

@vladmesh

Copy link
Copy Markdown
Owner

What

Candidate: branch pipeline/codegen-orchestrator-1707, commit 050c431 (tree 30202465d90f0ac5fa31c68e8a07f2f52a7efa06), one commit on main 7995ef2. No AI trailer.

Why

Carry the API's required GITHUB_WEBHOOK_SECRET through the existing production/stand deployment configuration and stand evidence redaction. This removes the known startup/configuration gap introduced by the authenticated completion receiver. It does not deploy, expose public ingress or activate a GitHub webhook.

How verified

CI runs Backend Docker-in-Docker Integration Tests, Build Service Image Candidates, Build Worker Base Image Candidates, CI Contract, Detect Changed Services, Fast Checks (Unit Tests), Lint (Ruff), Publish Service Image Release, Publish Worker Base Images, Required CI Gate, Service Image Entrypoint Imports on 050c431b71da; the merge waits for them to pass.

Card: codegen-orchestrator-1707

Deploy validates the GH_WEBHOOK_SECRET Environment secret as one safe token before any
host write and writes it into the server .env as GITHUB_WEBHOOK_SECRET. The stand
requires it for every suite before paid steps, renders it into the API environment,
and holds it as a protected value of both service-tail redactions and both artifact
admissions; a missing required redaction value fails the tail collection closed.
Other protected names keep their previous handling in the service-tail
redactor. The new webhook credential tests take the current UTC time
instead of a fixed calendar date.
@vladmesh
vladmesh merged commit f987901 into main Oct 18, 2026
28 checks passed
@vladmesh
vladmesh deleted the pipeline/codegen-orchestrator-1707 branch October 18, 2026 22:03
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.

1 participant