Skip to content

feat: enable firewall on claude and remove hooks - #4794

Closed
pelikhan with Copilot wants to merge 4 commits into
firewall-claudefrom
copilot/sub-pr-4792
Closed

feat: enable firewall on claude and remove hooks#4794
pelikhan with Copilot wants to merge 4 commits into
firewall-claudefrom
copilot/sub-pr-4792

Conversation

CopilotAI commented Nov 26, 2025

Copy link
Copy Markdown
Contributor

Enables AWF (Agent Workflow Firewall) support for Claude engine, replacing the legacy hooks-based network permissions system. Updates AWF to v0.4.0.

Changes

  • Claude engine now uses AWF binary for network access control (matching Copilot's implementation)
  • Removed deprecated Python hooks and settings.json based network permissions
  • Updated enableFirewallByDefault to use engine interface instead of string-based ID
  • Added GetClaudeAllowedDomains helper for Claude-specific domain handling
  • Updated documentation to reflect Claude's firewall support

Configuration

Both engines now use identical firewall configuration:

engine: claude # or copilotnetwork:
firewall: trueallowed:
- defaults
- python
- "api.example.com"

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Mossakaand others added 3 commits November 26, 2025 00:25
Signed-off-by: Jiaxiao (mossaka) Zhou <duibao55328@gmail.com>
Signed-off-by: Jiaxiao (mossaka) Zhou <duibao55328@gmail.com>
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
CopilotAI changed the title [WIP] Enable firewall on Claude and update AWF versionfeat: enable firewall on claude and remove hooksNov 26, 2025
CopilotAI requested a review from pelikhanNovember 26, 2025 00:46
@Mossaka
Mossakaforce-pushed the firewall-claude branch 3 times, most recently from d32156c to 505a16dCompareNovember 28, 2025 00:21
@pelikhan
pelikhan deleted the copilot/sub-pr-4792 branch December 1, 2025 03:45
Sign up for freeto 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.

3 participants

@pelikhan@Mossaka