Merge pull request #32 from mehalter/escape_url

fix: escape URL string to handle special characters
rkyleg · Nov 26, 2023 · 08ddbff · 08ddbff
2 parents 1788f0f + cf9e92a
commit 08ddbff
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/bin/list-all b/bin/list-all
@@ -13,12 +13,11 @@ releases_path="https://api.github.com/repos/JuliaLang/julia/releases?page="
 page=0
 
 if [ -n "$GITHUB_API_TOKEN" ]; then
-  cmd="$cmd -H 'Authorization: token $GITHUB_API_TOKEN' $releases_path"
+    cmd="$cmd -H 'Authorization: token $GITHUB_API_TOKEN'"
 fi
-cmd="$cmd $releases_path"
 
 while (( ++page == 1 )) || grep -q 'rel="next"' $headers; do
-    eval $cmd$page >> $releases
+    eval "$cmd '$releases_path$page'" >>$releases
 done
 
 versions=($(sed -n 's/.*"tag_name": "\(.*\)",/\1/p' $releases | sed 'y/v/ /' | sort_versions))