Commit 086cac4
committed
ci(common): sandbox Claude Code behind Squid proxy + iptables
Run the claude-code-action inside a network sandbox to prevent
data exfiltration to unauthorized hosts. Two layers of defense:
- Squid proxy: L7 domain allowlist (.anthropic.com, .github.com, etc.)
- iptables: blocks direct outbound TCP from the runner UID
All dependencies (Bun, action node_modules, Claude Code CLI, OIDC
token exchange) are pre-installed before lockdown because the action's
internal installers use fetch() which ignores HTTP_PROXY.
Also switches from --allowedTools to --dangerously-skip-permissions
since the network sandbox handles security at the infrastructure level.
update claude file with proper container setup
fix: shellchecks
fix zizmor warning
ci(claude): rewrite workflow from template, address PR #1995 security review
- Drop action wrapper, run claude CLI directly (avoids MCP stdin blocking)
- Remove dead pull_request trigger
- Separate GH_TOKEN from system prompt construction step
- Tighten iptables: resolve Squid IP dynamically, block UDP/ICMP
- Restrict squid allowlist to 3 domains (api.anthropic.com, platform.claude.com, github.com)
- Cache Squid Docker image, add iptables save/restore cleanup
- Add tracking comment for run visibility
- Fix token revocation to use HTTPS_PROXY
fix: replace A && B || C with proper if-then-else (SC2015)
fix: capture error details instead of silent suppression
OIDC exchange and token revocation now log the server response
on failure instead of swallowing it with -sf/--silent/2>/dev/null.
fix: shellcheck SC2001 and SC2015 in claude workflow
Replace sed prompt extraction with parameter expansion (SC2001).
chore: harden security practices
chore: update claude action from secutiry1 parent 974fe41 commit 086cac4
File tree
2 files changed
+563
-121
lines changed- .github
- squid
- workflows
2 files changed
+563
-121
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
0 commit comments