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

Patch 1 #4

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

Patch 1 #4

wants to merge 4 commits into from

Conversation

tim0n3
Copy link

@tim0n3 tim0n3 commented Dec 15, 2021

Please, may you review and merge if you agree with the changes?

Hope the commit notes and comments suffice.

Added  flush & clear for the raw table rules;
Added verbose output to the terminal for visual aesthetic and ease-of-understanding which tables' rules you're looking at;
Added listing of all rules (numbered) for each table in the firewall;
Added details to the comment regarding the rule for allowing traffic on the lo iface;
Added discriptive comment to the rule for iface = lo so that when you list the rule using iptables -nvL you can see comments;
Added to the comment regarding the default policy for the filter tables chains;
Replaced old style "-m state --state" with newer "-m state --state";
Moved filter table chain policy definition to the bottom after all the rules have been added to prevent host lockout (if running the script remotely);
Added comment to drop rule on the INPUT chain for better desciption;
Added comments to EST,REL rules on INPUT and OUTPUT chains;
Added rules for ping counters (new) for both in/out conns. You only need NEW state rule as your first rules for EST,REL conns will handle the rest of the connection;
Added DDoS/Portscan rules in RAW and MANGLE tables;
Added listing of all the rules at the bottom of the script;
Added text to display what tables rules are being edited;
Explicitly specified filter table for the flush/clear rules commands at the top.
Added  flush & clear for the raw table rules;
Added verbose output to the terminal for visual aesthetic and ease-of-understanding which tables' rules you're looking at;
Added listing of all rules (numbered) for each table in the firewall.
Added text to state what's happening;
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