phusion / passenger_apt_automation

Tools for automatically building a Debian APT repository for Phusion Passenger
MIT License
20 stars 20 forks source link

Replace wget with cURL #16

Closed shaneog closed 9 years ago

shaneog commented 9 years ago

wget cannot connect to the get.docker.com HTTPS URL - it fails with a HTTPS error.

curl works fine.

Error I received:

--2015-09-15 18:09:58--  https://get.docker.com/
Resolving get.docker.com (get.docker.com)... 54.230.59.225, 54.192.59.20, 54.230.58.52, ...
Connecting to get.docker.com (get.docker.com)|54.230.59.225|:443... connected.
OpenSSL: error:14077410:SSL routines:SSL23_GET_SERVER_HELLO:sslv3 alert handshake failure
Unable to establish SSL connection.
FooBarWidget commented 9 years ago

I have nothing against using curl instead of wget, but you really should use curl -sSL instead of curl -s.

shaneog commented 9 years ago

Done in dbf623f082d3d994916bb7bcfd2ce296ad921737

FooBarWidget commented 9 years ago

Thanks, I've merged this change.