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

Setup fail2ban to cover wings.md #468

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

Conversation

Chronic-Reflexes
Copy link

Hmm yes security tutorial

Recently did it for myself and wanted to share it with the community as any layer of security is a good thing to have.

Found the filter online linked here: https://github.com/Chewbaka69/fail2ban_pterodactyl

Hmm yes security tutorial

Recently did it for myself and wanted to share it with the community as any layer of security is a good thing to have.

Found the filter online linked here:  https://github.com/Chewbaka69/fail2ban_pterodactyl
Copy link
Contributor

@devnote-dev devnote-dev left a comment

Choose a reason for hiding this comment

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

Overall not bad, I've pointed out the main points to consider changing but you may also want to check the punctuation. Good work though 👍

community/Wings Fail2ban Setup.md Show resolved Hide resolved
community/Wings Fail2ban Setup.md Outdated Show resolved Hide resolved
community/Wings Fail2ban Setup.md Outdated Show resolved Hide resolved
community/Wings Fail2ban Setup.md Outdated Show resolved Hide resolved
community/Wings Fail2ban Setup.md Outdated Show resolved Hide resolved
@Chronic-Reflexes
Copy link
Author

ive made it short and sweet, its my first time trying to do markdown with github, so thank you for getting me to go it the right way, turns out its much easier than adding quad spaces every line 😄

@Jayryn
Copy link

Jayryn commented Jul 23, 2022

I am currently not on pc to make a commit but you missed the second L on install in the step 1 comment. (Just noticed this)

Overall I'm happy to see someone working for this as I would be happy to implement it. Great work.

@@ -2,33 +2,33 @@

Fail2ban is a program that works with iptables to better secure your server and services from brute force login attempts

### Step 1: if you havent already Using your preferred package manager instal Fail2Ban
### Step 1: instal Fail2Ban
Copy link

Choose a reason for hiding this comment

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

It should be install here.

Copy link

@Jayryn Jayryn left a comment

Choose a reason for hiding this comment

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

Fix typo in Step 1

@trenutoo
Copy link
Member

Thanks for taking the time to contribute to the project. The content itself is fine, but the wording/grammar needs some work. I can go through it at some point later if I have time.

@trenutoo trenutoo added the Please Correct Waiting for user to fix the requested issues label Oct 14, 2022

### Step 4: Create the filter file and copy some more text

`cd /etc/fail2ban/conf.d/`

Choose a reason for hiding this comment

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

By my side, on Ubuntu 22.04 and Fail2ban 0.11.2-6, the filter file comes into /etc/fail2ban/filter.d/
https://help.ubuntu.com/community/Fail2ban

failregex = failed to validate user credentials \([^\)]+\) ip=<HOST>:.* subsystem=sftp username=.*$

ignoreregex =

Choose a reason for hiding this comment

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

As far as I understand, in order to increase the search performance from the systemd backend, it is great to add a journalmatch. Anyway, without it, I had a warning that told me this parameter was missing. Here is what I have done by my side: journalmatch = _SYSTEMD_UNIT=wings.service + _COMM=wings

@danny6167 danny6167 self-assigned this Sep 11, 2024
@rowansc1
Copy link
Contributor

Any updates on this PR? Seems like a good thing to have!

@danny6167
Copy link
Member

I was intending to test and finish this PR, but my time has been drawn elsewhere for the moment.
I'll un-assign my self in case someone else wants to take this up.

@danny6167 danny6167 removed their assignment Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Please Correct Waiting for user to fix the requested issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants