Thanks for the PR @evolutionxbox! The https is not too relevant here (since we're just checking if we're online), but I'm not against switching to https.

Why did you disable verification using -k though?

If it's getting switched to https, it'd be better to use -3 since both domains support TLSv1.3.
-k just makes the connection insecure.

First of all wget --spider seems optimised for this task. wget isn't packaged by default on MacOS, so it's probably best to stick to curl.

• -3 uses SSlv3 (insecure, according to the manage) It's better to use default TLS. There isn't really a good reason for -k to be there either.
• -s is a minor optimisation.
• I assume the update script cares whether Github works server-side, so -fL should be added there.

Yes, meant to say --tlsv1.3

@Tokarak yes the intent of those calls is just a naive "are we online" check.

Sounds like removing the -k is the most important bit, the rest doesn't really matter.

It could even just be a curl -I since all I care about is whether we can talk to the url.

@evolutionxbox, could you change the -k to -I (that's an i, not an L) so I can merge this?

@evolutionxbox could you change the -k to -I so I can merge? I don't mind doing it myself but want you to have the props on your github profile for your contribution :)

@actuallymentor I didn't mean to ignore the comments. I have updated the PR ✅