Skip to content

docs: Fail2Ban - Add example with required ENV to enable#4402

Merged
georglauterbach merged 2 commits intodocker-mailserver:masterfrom
Lasslos:patch-1
Mar 6, 2025
Merged

docs: Fail2Ban - Add example with required ENV to enable#4402
georglauterbach merged 2 commits intodocker-mailserver:masterfrom
Lasslos:patch-1

Conversation

@Lasslos
Copy link
Copy Markdown
Contributor

@Lasslos Lasslos commented Mar 5, 2025

Added instructions to edit env to enable fail2ban

Description

Docs are missing a hint on how to actually enable fail2ban

Fixes #

Added hint.

Lasslos and others added 2 commits March 5, 2025 14:00
Added instructions to edit env to enable fail2ban
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 5, 2025

Documentation preview for this PR is ready! 🎉

Built with commit: 0a91da1

@polarathene polarathene changed the title Update docs to help with Fail2Ban instruction docs: Fail2Ban - Add example with required ENV to enable Mar 5, 2025
@polarathene polarathene added kind/improvement Improve an existing feature, configuration file or the documentation area/documentation labels Mar 5, 2025
@polarathene polarathene added this to the v15.0.1 milestone Mar 5, 2025
Copy link
Copy Markdown
Member

@polarathene polarathene left a comment

Choose a reason for hiding this comment

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

Direct preview page link: https://pullrequest-4402--dms-doc-previews.netlify.app/config/security/fail2ban


Thanks for the contribution, I've revised it a fair bit as you can see, but it should communicate the required config clearly and provides a bit more context on the warning/bug sections.

I've added mention of NET_RAW, but I'm not sure how relevant that is for DMS so I've not added it's usage in the example above (which may confuse the reader a little bit. The referenced LSIO image documents the purpose of NET_RAW with their image.

F2B docs seem a bit light regarding capabilities required. I've opened this discussion to seek clarity on that.

@georglauterbach georglauterbach merged commit 6b1a566 into docker-mailserver:master Mar 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/documentation kind/improvement Improve an existing feature, configuration file or the documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants