Skip to content

added some options#23

Open
greatquux wants to merge 18 commits intoEvanAnderson:masterfrom
greatquux:master
Open

added some options#23
greatquux wants to merge 18 commits intoEvanAnderson:masterfrom
greatquux:master

Conversation

@greatquux
Copy link

hi - i made a couple changes which work for me
1 - because i'm disabling advanced f/w i wanted to use route method on newer versions of windows, so i created registry entries to set this as a preference (didn't modify the ADM)
2 - i found that using an IP of 0.0.0.0 as my blackhole IP worked (seems insane of course, but try it yourself and you'll see!) so I made that the default in case it's not defined by the other methods
3 - i changed the route delete command to only refer to the blacklisted IP since this works and the full command doesn't work if I use 0.0.0.0 as my blackhole IP

greatquux and others added 5 commits March 30, 2017 13:48
these extra arguments are not necessary and can cause unblocking to fail depending on blackholeip used
I have tested using a blackhole IP of 0.0.0.0 on 2008/2008R2/2012/2012R2 and it works in each case to effectively deny network access to the offending IP.
@greatquux
Copy link
Author

also just added a whitelist registry entry

- whitelist will now allow basic subnet matching (ie, including 1.2.3 in the whitelist will match IPs from 1.2.3.1-254)
- don't block any of the IPs on the local network cards; I have also observed this can sometimes happen in practice and leads to very strange networking issues!
add logging when not blocking system, whitelisted, or NIC IPs
due to Option Explicit script wasn't actually working after yesterday's changes
add options for new settings
@greatquux
Copy link
Author

this script has really been working well so far, can't believe this sort of thing is not built in

Add Trim() when reading the whitelist, if there is trailing whitespace it will cause all IPs to get whitelisted!
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