Skip to content
Snippets Groups Projects

.automated_script.sh: add proper HTTP status checking

Merged Dariusz Pelowski requested to merge darekp/archiso:master into master
All threads resolved!

When a download attempt fails with an HTTP error, the body of the failed request will be written to the output file with a 0 exit status, causing the execution of an error message as a script.

This issue is resolved by adding the --fail option to curl.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Thanks for your contribution!

    Please adjust the changelog entry and this is good to go :)

  • David Runge requested changes

    requested changes

  • Dariusz Pelowski resolved all threads

    resolved all threads

  • added 1 commit

    • 6dd106b2 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • nl6720 changed milestone to %v79

    changed milestone to %v79

  • assigned to @dvzrv and @nl6720

  • nl6720 approved this merge request

    approved this merge request

  • David Runge approved this merge request

    approved this merge request

  • nl6720 added 1 commit

    added 1 commit

    • 75e57093 - .automated_script.sh: add proper HTTP status checking

    Compare with previous version

  • nl6720 resolved all threads

    resolved all threads

  • nl6720 mentioned in commit 57589df1

    mentioned in commit 57589df1

  • nl6720 merged manually

    merged manually

  • Please register or sign in to reply
    Loading