Skip to content

Pr716 - #721

Merged
msimerson merged 6 commits into
masterfrom
pr716
Aug 14, 2026
Merged

Pr716#721
msimerson merged 6 commits into
masterfrom
pr716

Conversation

@msimerson

@msimerson msimerson commented Aug 13, 2026 •

Copy link
Copy Markdown
Owner

Fixes closes #716

@msimerson
msimerson marked this pull request as ready for review August 13, 2026 23:58
@msimerson
msimerson requested a balanced review from Copilot August 13, 2026 23:58

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Hardens jail host configuration and adds IPv6 syslog support.

Changes:

  • Stops copying the host’s /etc/hosts into the base jail.
  • Adds minimal DNS, syslog, and BSD cache host entries.
  • Configures syslogd to bind IPv6 when available and adds tests.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
Changes.md Documents host and syslog updates.
include/network.sh Generates minimal jail host entries.
mail-toaster.sh Installs entries into staged jails.
provision/base.sh Stops copying the host’s hosts file.
provision/host.sh Adds conditional IPv6 syslog binding.
test/mail-toaster.bats Tests minimal hosts generation.
test/provision/host.bats Tests syslog address-family flags.
test/provision/stubs/mail-toaster.sh Adds the new provisioning stub.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread include/network.sh
Comment thread provision/host.sh
@msimerson
msimerson merged commit bb90278 into master Aug 14, 2026
4 checks passed
@msimerson
msimerson deleted the pr716 branch August 14, 2026 00:24
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.

3 participants