Skip to content

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

Closed
migmartri wants to merge 1 commit into
chainloop-dev:mainfrom
migmartri:sandbox-kit-download-egress
Closed

migmartri wants to merge 1 commit into
chainloop-dev:mainfrom
migmartri:sandbox-kit-download-egress

Conversation

@migmartri

@migmartri migmartri commented Aug 31, 2026 •

Copy link
Copy Markdown
Member

Adds download.chainloop.dev to the egress allowlist of the Chainloop-traced Claude Code sandbox kit (devel/sandbox-kit/spec.yaml), so the EE CLI binary download is not blocked by the sandbox's deny-by-default network policy.

AI disclosure: this change was produced with the assistance of Claude Code.

Review in cubic

Add download.chainloop.dev to the traced sandbox kit's egress allowlist so
the EE CLI binary download is not blocked by the deny-by-default network
policy.

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

Chainloop-Trace-Sessions: 0f5545f7-2d2a-42b6-ac77-2761b445a17a, ce7edc24-1767-49b5-b2ab-71fbefb0d4d2, e4e8a764-7ec9-4a9f-98ac-f8fe7c21b9f2
@chainloop-platform

chainloop-platform Bot commented Aug 31, 2026 •

Copy link
Copy Markdown
Contributor

AI Session Checks — ⚠️ 3 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 #3390 does not reference a user story or issue in title, description, or branch 'feat(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

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 1 file

Heads up: you’re close to your included review allowance. Set a flex budget so reviews don’t pause.

Re-trigger cubic

@migmartri migmartri closed this Aug 31, 2026
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