search

radekg / terraform-provisioner-ansible

Ansible with Terraform 0.14.x
Apache License 2.0
572 stars 100 forks source link

update the deploy-release.sh script #161

Closed sherzberg closed 3 years ago

sherzberg commented 3 years ago

Summary

curl -i is not returning a status: header, it is now HTTP/2, so the deploy-release.sh script is not working anymore.

Before this change, the script would just exit 1 with no log messages saying what happened and the release was not downloaded.

Here is some of the output if you make the curl request:

$ curl -i https://github.com/radekg/terraform-provisioner-ansible/releases/download/v2.5.0/terraform-provisioner-ansible-linux-amd64_v2.5.0
HTTP/2 302 
server: GitHub.com
date: Tue, 30 Mar 2021 18:23:39 GMT
content-type: text/html; charset=utf-8
vary: X-PJAX, Accept-Encoding, Accept, X-Requested-With
...<removed>...
radekg commented 3 years ago

This is awesome. Thank you very much, @sherzberg. In the next few days, I'll bump Terraform dependency to 0.15 so this will be released together with that update.