[PR #18] [MERGED] docker: improve safety, UX, and Metasploit MCP defaults #22

Closed
opened 2026-02-27 13:59:27 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/GH05TCREW/pentestagent/pull/18
Author: @giveen
Created: 1/15/2026
Status: Merged
Merged: 1/15/2026
Merged by: @GH05TCREW

Base: mainHead: docker/mcp-docker-changes


📝 Commits (2)

  • 3635077 docker: improve safety, UX, and Metasploit MCP defaults
  • 0b40471 chore: remove PR_BODY.md

📊 Changes

6 files changed (+81 additions, -16 deletions)

View changed files

.dockerignore (+17 -0)
📝 docker-compose.yml (+4 -1)
📝 docker-entrypoint.sh (+25 -12)
📝 pentestagent/interface/main.py (+9 -0)
📝 pentestagent/interface/tui.py (+15 -3)
📝 scripts/setup.sh (+11 -0)

📄 Description

Why:

  • Persist artifacts to /app/loot so container outputs remain available when mounted.
  • Avoid mandatory host chown; make chown opt-in via CHOWN_ON_START to prevent accidental ownership changes.
  • Bind msfrpcd to 127.0.0.1 by default and add EXPOSE_MSF_RPC opt-in to avoid exposing RPC to host network.
  • Replace crashing assertion on missing default model with a friendly CLI/TUI error path.
  • Add .dockerignore to reduce build context and avoid copying unnecessary files.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/GH05TCREW/pentestagent/pull/18 **Author:** [@giveen](https://github.com/giveen) **Created:** 1/15/2026 **Status:** ✅ Merged **Merged:** 1/15/2026 **Merged by:** [@GH05TCREW](https://github.com/GH05TCREW) **Base:** `main` ← **Head:** `docker/mcp-docker-changes` --- ### 📝 Commits (2) - [`3635077`](https://github.com/GH05TCREW/pentestagent/commit/36350775f99baa143aa8ca4682f356c636e0ee59) docker: improve safety, UX, and Metasploit MCP defaults - [`0b40471`](https://github.com/GH05TCREW/pentestagent/commit/0b4047131043c25a94980e74cb69662ad1ef8ae7) chore: remove PR_BODY.md ### 📊 Changes **6 files changed** (+81 additions, -16 deletions) <details> <summary>View changed files</summary> ➕ `.dockerignore` (+17 -0) 📝 `docker-compose.yml` (+4 -1) 📝 `docker-entrypoint.sh` (+25 -12) 📝 `pentestagent/interface/main.py` (+9 -0) 📝 `pentestagent/interface/tui.py` (+15 -3) 📝 `scripts/setup.sh` (+11 -0) </details> ### 📄 Description Why: - Persist artifacts to /app/loot so container outputs remain available when mounted. - Avoid mandatory host chown; make chown opt-in via CHOWN_ON_START to prevent accidental ownership changes. - Bind msfrpcd to 127.0.0.1 by default and add EXPOSE_MSF_RPC opt-in to avoid exposing RPC to host network. - Replace crashing assertion on missing default model with a friendly CLI/TUI error path. - Add .dockerignore to reduce build context and avoid copying unnecessary files. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-27 13:59:27 -05:00
yindo closed this issue 2026-02-27 13:59:29 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: yindo/pentestagent#22