FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

docs: Fail2Ban - Add example with required ENV to enable by Lasslos · Pull Request #4402 · docker-mailserver/docker-mailserver · GitHub

/ docker-mailserver Public

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

Merged
georglauterbach merged 2 commits into
docker-mailserver:masterfrom
Lasslos:patch-1
Mar 6, 2025
ConversationCommits (2)ChecksFiles changed
Merged

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

Conversation

Lasslos commented Mar 5, 2025

Copy link
Copy Markdown
Contributor

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
Update fail2ban.md
402ffc5 
Added instructions to edit env to enable fail2ban
Update fail2ban.md
0a91da1

github-actions Bot commented Mar 5, 2025

Copy link
Copy Markdown
Contributor

Documentation preview for this PR is ready! 🎉

Built with commit: 0a91da1

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

polarathene left a comment •
edited
Loading

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

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 merged commit 6b1a566 into docker-mailserver:master Mar 6, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

casperklein casperklein approved these changes
polarathene polarathene approved these changes
georglauterbach georglauterbach approved these changes

Assignees

No one assigned

Labels

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

Projects

None yet

Milestone

v15.0.1

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL