Skip to content

chore(devel): allow download.chainloop.dev egress in the sandbox kit - #3389

Closed
migmartri wants to merge 1 commit into
mainfrom
chore/sandbox-kit-allow-download-host
Closed

migmartri wants to merge 1 commit into
mainfrom
chore/sandbox-kit-allow-download-host

Conversation

@migmartri

@migmartri migmartri commented Aug 31, 2026 •

Copy link
Copy Markdown
Member

Adds download.chainloop.dev:443 to the traced Docker sandbox kit's network allowlist (devel/sandbox-kit/spec.yaml), alongside the existing azurefd CDN origin, so the EE CLI installer can fetch binaries through the vanity download host.

AI disclosure: this change was produced with the assistance of Claude Code; the commit carries an Assisted-by: Claude Code trailer.

Review in cubic

Add the vanity EE CLI download host to the traced sandbox kit's network
allowlist so the CLI installer can fetch binaries through it in addition
to the azurefd origin.

Assisted-by: Claude Code
Signed-off-by: Miguel Martinez Trivino <miguel@chainloop.dev>

Chainloop-Trace-Sessions: 0f5545f7-2d2a-42b6-ac77-2761b445a17a, e4e8a764-7ec9-4a9f-98ac-f8fe7c21b9f2
@chainloop-platform

chainloop-platform Bot commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

AI Session Checks — ⚠️ 2 session(s) missing

Missing AI Coding Sessions

We detected commits in this PR that were AI-assisted, but the matching Chainloop Trace session(s) could not be found in Chainloop.

Please make sure the AI coding session evidence has been sent by the Chainloop CLI, or add the skip-ai-session label to this PR to bypass this check.

Learn more about Chainloop Trace.


Security Checks — ⚠️ 1 failing

✅ secret-scan

Status Policy Messages
✅ Passed secrets-detection -

✅ sast-scan

Status Policy Messages
✅ Passed owasp-top10-2025 -
✅ Passed sast -
✅ Passed cwe-top25 -
✅ Passed cwe-top26-40-cusp -

✅ iac-scan

Status Policy Messages
✅ Passed iac-misconfiguration -

PR info

Status Policy Messages
✅ Passed pr-description-required -
⚠️ Failed pr-user-story-linked PR/MR #3389 does not reference a user story or issue in title, description, or branch 'chore(devel): allow download.chainloop.dev egress in the sandbox kit'. Expected patterns: ["(?i)[A-Z]+-[0-9]+", "#[0-9]+", "(?i)[A-Z]{2", "}-[0-9]+", "(?i)gh-[0-9]+", "(?i)\[[A-Z]+-[0-9]+\]"]

⏭️ 2 scans not applied

Scan Reason
vulnerability-scan no manifest/lockfile changed
github-actions-scan no workflow files changed

View attestation ↗


Powered by Chainloop and Chainloop Trace

@migmartri migmartri closed this Aug 31, 2026
@migmartri
migmartri deleted the chore/sandbox-kit-allow-download-host branch August 31, 2026 17: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