diff options
| author | Ben Busby <contact@benbusby.com> | 2022-07-31 11:47:28 -0600 |
|---|---|---|
| committer | Ben Busby <contact@benbusby.com> | 2022-07-31 11:47:28 -0600 |
| commit | 3460986481b11cbd212c0504db49d9684ea9eeeb (patch) | |
| tree | 16a61a4eed9c751742745b255ee517e76a471b12 /.github | |
| parent | 431ea613d19fc471de14629dc5dcc7fd22cfbddd (diff) | |
| download | farside-3460986481b11cbd212c0504db49d9684ea9eeeb.tar.gz | |
Handle dig return code 9 in update workflow
If dig returns exit code 9 (no reply from server) when checking an
instance for cloudflare records, it shouldn't fail the CI build but
rather just skip adding the instance to the non-cloudflare services
list.
This should be re-evaluated soon to see if the CI build routinely has
issues with getting a server reply. If so, a different approach needs to
be taken to check if an instance is using cloudflare.
Diffstat (limited to '.github')
| -rw-r--r-- | .github/workflows/update-instances.yml | 20 |
1 files changed, 11 insertions, 9 deletions
diff --git a/.github/workflows/update-instances.yml b/.github/workflows/update-instances.yml index 3eec4a9..c77c551 100644 --- a/.github/workflows/update-instances.yml +++ b/.github/workflows/update-instances.yml @@ -32,17 +32,19 @@ jobs: file="services-full.json" while read -r line; do - if [[ "$line" == "\"https://"* ]]; then - domain=$(echo "$line" | sed -e "s/^\"https:\/\///" -e "s/\",//" -e "s/\"//") - ns=$(dig ns "$domain") - if [[ "$ns" == *"cloudflare"* ]]; then - echo "\"$domain\" using cloudflare, skipping..." - else - echo "$line" >> out.json - fi + if [[ "$line" == "\"https://"* ]]; then + domain=$(echo "$line" | sed -e "s/^\"https:\/\///" -e "s/\",//" -e "s/\"//") + ns=$(dig ns "$domain" || true) + if [[ "$ns" == *"cloudflare"* ]]; then + echo "\"$domain\" using cloudflare, skipping..." + elif [ ${#ns} -eq 0 ]; then + echo "Unable to verify records for \"$domain\", skipping..." else - echo "$line" >> out.json + echo "$line" >> out.json fi + else + echo "$line" >> out.json + fi done <$file # Remove any trailing commas from new instance lists |