Fix broken syntax in cloudflare template
authorMatt Palmer <mpalmer@hezmatt.org>
Fri, 30 Oct 2015 21:56:33 +0000 (08:56 +1100)
committerMatt Palmer <mpalmer@hezmatt.org>
Fri, 30 Oct 2015 21:56:33 +0000 (08:56 +1100)
My assumption is that CF used to send their IP list without a trailing
newline, and they've recently fixed that, causing riking's hack to emit an
*extra* newline, which caused the reported problem.

templates/cloudflare.template.yml

index 6d5e95338e09413dd5f441be40431838fb411cd0..df8a00d06172de4c7f3ab8cc229b96e757c4de4b 100644 (file)
@@ -6,7 +6,6 @@ run:
         #!/bin/bash -e
         # Download list of CloudFlare ips
         wget https://www.cloudflare.com/ips-v4 -O - > /tmp/cloudflare-ips
-        echo >> /tmp/cloudflare-ips
         wget https://www.cloudflare.com/ips-v6 -O - >> /tmp/cloudflare-ips
         # Make into nginx commands and escape for inclusion into sed append command
         CONTENTS=$(</tmp/cloudflare-ips sed 's/^/set_real_ip_from /' | sed 's/$/;/' | tr '\n' '\\' | sed 's/\\/\\n/g')