Skip to content

ci: allow github.com:443 in dependency-review egress#482

Merged
BenJule merged 1 commit into
masterfrom
fix/dependency-review-egress
Jun 6, 2026
Merged

ci: allow github.com:443 in dependency-review egress#482
BenJule merged 1 commit into
masterfrom
fix/dependency-review-egress

Conversation

@BenJule

@BenJule BenJule commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Closes #481

The Review dependency changes job blocks all egress except api.github.com:443, but actions/checkout needs github.com:443 for its git-fetch → checkout fails on every PR (#476, #480). Adds github.com:443 to the allowed-endpoints. Workflow-only change — no build impact.

@github-actions github-actions Bot added the area: ci GitHub Actions, CI/CD label Jun 6, 2026
@github-actions github-actions Bot added this to the v02.07.00-dev milestone Jun 6, 2026
The 'Review dependency changes' checkout failed to reach github.com even
with github.com:443 allow-listed ('domain not allowed: github.com'). The
list used a literal block (|) => newline-separated, which harden-runner
did not honour as separate endpoints. Switch to a folded block (>)
(whitespace-separated) and add github.com:443 — matching the working
detect_changes job in cd-nightly.
@BenJule BenJule force-pushed the fix/dependency-review-egress branch from 0c21c1a to a5a6891 Compare June 6, 2026 04:14
@BenJule BenJule merged commit 6c8c075 into master Jun 6, 2026
3 checks passed
@BenJule BenJule deleted the fix/dependency-review-egress branch June 6, 2026 04:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ci GitHub Actions, CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: dependency-review checkout fails — github.com blocked by Harden-Runner egress

1 participant