Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(action): Print get.docker.com error messages #159

Merged
merged 1 commit into from
Oct 27, 2022
Merged

Conversation

Kurt-von-Laven
Copy link
Contributor

Replace --fail with --fail-with-body in cURL invocation that downloads the rootless Docker install script. This causes cURL to display any error message it may receive from the server, which is often helpful context to have when debugging.

Replace --fail with --fail-with-body in cURL invocation that downloads
the rootless Docker install script. This causes cURL to display any
error message it may receive from the server, which is often helpful
context to have when debugging.
Copy link
Contributor

@mwarres mwarres left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Kurt-von-Laven Kurt-von-Laven merged commit 4ec6d1b into main Oct 27, 2022
@Kurt-von-Laven Kurt-von-Laven deleted the action branch October 27, 2022 02:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants