MattiSG / adblock

Block ads and trackers system-wide, before a request is even issued.
MIT License
175 stars 13 forks source link

Use secure URL #13

Closed eddie-zhang closed 6 years ago

eddie-zhang commented 6 years ago

Using curl with the insecure URL (http://) now produces a 302 error with this body:

<!DOCTYPE HTML PUBLIC "-//IETF//DTD HTML 2.0//EN">
<html><head>
<title>302 Found</title>
</head><body>
<h1>Found</h1>
<p>The document has moved <a href="https://someonewhocares.org/hosts/zero/hosts">here</a>.</p>
</body></html>

Using the secure URL ensures this HTML is not written to the hosts file.

MattiSG commented 6 years ago

Hey @eddie-zhang! Thank you so much for the heads up on this change. Sorry it took me so long to merge, the change was actually slightly more complex as lwp-request I used for the modification time check did not support HTTPS HEAD. I switched to curl and published the change as v1.1.0.

Thanks again! 👍