Debloat-Windows-10 icon indicating copy to clipboard operation
Debloat-Windows-10 copied to clipboard

Recommendation: Move the DNS/IPs out of hardcoded variables and into their own separate text files

Open yllekz opened this issue 2 years ago • 1 comments

I'm not sure if this repo is maintained anymore, but just from a QoL/best practice perspective, I found it interesting that the server names and IPs were hardcoded into the PowerShell scripts. This makes it difficult to read the raw data if one wanted to say, import the list into Pi-Hole and block the servers from the network. From a maintenance perspective, it also separates the script's functional portions from the ephemeral data (i.e., the ever-changing servers/IPs).

Could it be possible to move the server names/IPs specified in the debloat ps1 files to their own separate and sanitized text files?

yllekz avatar May 18 '22 20:05 yllekz

It's only barely maintained at this point since I lack the time to adapt existing code and test new additions.

It's certainly possible to read the ip/domain lists from an external file, next to the scripts.

If somebody gives me a PR that adjusts the code accordingly, I'd be happy to merge it.

W4RH4WK avatar May 19 '22 07:05 W4RH4WK