Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fail2ban whitelist #2444

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

Gofannon
Copy link
Contributor

Problem

  • error in documentation made a nextcloud upgrade to fail (as fail2ban was broken, the upgrade failed)

Solution

  • fix the doc
  • proposal to enhance it

Solution long term

why not integrate the file yunohost-whitelist.conf in the "core" so people don't have to create it by hand . Could it be doable and where/how add it to the backlog ?

I have no idea of the "complexity" of it. Here is how I see it:

  1. create the file
    1. copy the relevent part of fail2ban/jail.conf or take the one of this doc
    2. Create file https://github.com/YunoHost/yunohost/blob/dev/conf/fail2ban/yunohost-whitelist.conf
  2. Add it to the "core"
    1. https://github.com/YunoHost/yunohost/blob/dev/hooks/conf_regen/52-fail2ban#L18
  3. Problems
    1. how to manage existing whitelist config files and content ?
    2. Upgrades by YunoHost ?
  • And how do you relevantly fix that problem

PR checklist

  • I'm not doing a PR for an application, I promise, I know that this kind of changes must go directly into the app packages themselves
  • PR finished and ready to be reviewed

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.

1 participant