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 the curl pipe'ed to bash docs to work with the ssh-host.sh changes. #276

Merged
merged 1 commit into from
Oct 16, 2023

Conversation

jdoss
Copy link
Contributor

@jdoss jdoss commented Oct 16, 2023

This addresses the issues when using curl to pipe the file directly to bash. This due to how a pipe works. The bash script uses read which in a piped use case reads from standard input (the pipe), but the shell already read all the standard input so there isn't anything for the read to read.

This change uses <() which will store curl command as a temp file and executes it with bash.

@jdoss jdoss requested a review from a team as a code owner October 16, 2023 15:52
@jdoss jdoss merged commit b8c886e into main Oct 16, 2023
3 checks passed
@jdoss jdoss deleted the jdoss/fix_ssh_docs branch October 16, 2023 19:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants