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

Consolidate firewall rules into a single rule? #79

Open
Gittyperson opened this issue Oct 16, 2017 · 6 comments
Open

Consolidate firewall rules into a single rule? #79

Gittyperson opened this issue Oct 16, 2017 · 6 comments

Comments

@Gittyperson
Copy link

Gittyperson commented Oct 16, 2017

Don't know if there's any impact on system performance (probably not), but is there a reason for creating separate rules (more than 150) instead of merging them into one? I believe Windows Firewall supports this and ranges can be entered too:

64.4.23.0-64.4.23.255,65.52.100.0-65.52.100.255, 191.239.54.52,207.46.41.202,207.68.166.254 etc.
Just a single, large line into "Remote address".

If anything, it will look better in the rules list. Perhaps the current method is preferred because separate rules are easier to enable/disable for testing purposes.

@crazy-max
Copy link
Owner

Multiple entries can be specified for remoteip by separating them with a comma according to the Netsh AdvFirewall Firewall Commands but like you said it's easier to separate rules for testing purposes.
I don't know if network performances can be altered but this is not significant on a personal computer but can be on a router.
We can try to benchmark to have an idea.

@Atavic
Copy link

Atavic commented Oct 18, 2017

The improvement should be minimal, beyond human recognition, in the scale of milliseconds.

@crazy-max
Copy link
Owner

After some investigation there is no impact on system performance for firewall and hosts rules.

@Gittyperson
Copy link
Author

Gittyperson commented Nov 22, 2017

By the way, it seems that adding multiple rules (on the Windows Firewall) takes a lot longer. For example, I just added the new rules with WindowsSpyBlocker and it took around a minute. Adding a single (merged) rule would probably require a couple of seconds.

@crazy-max
Copy link
Owner

You right @Gittyperson I will make an improvement on this

@crazy-max crazy-max reopened this Nov 22, 2017
@crazy-max crazy-max self-assigned this Nov 22, 2017
@Atavic
Copy link

Atavic commented Apr 9, 2018

My previous statement was about Linux OS, not Windows. Sorry for that.

@crazy-max crazy-max removed their assignment Jun 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants