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

support curl, wget, perl timeout replacement, busybox timeout params #17

Closed
wants to merge 2 commits into from
Closed

support curl, wget, perl timeout replacement, busybox timeout params #17

wants to merge 2 commits into from

Conversation

DukeyToo
Copy link

Will fix #15, #16, #5 .

Adds command-line switches for busybox, and using curl and wget for more correct tests of http endpoints. If timeout command is not found, then will attempt to use a perl replacement.

These are all changes I made for my own needs in orchestrating the startup and configuration of a bunch of docker containers.

@dakl
Copy link

dakl commented Jul 2, 2016

👍

@dequis
Copy link

dequis commented Aug 12, 2016

# sh wait-for-it.sh
wait-for-it.sh: line 67: syntax error: unexpected "(" (expecting "fi")

BusyBox v1.24.2 (from docker:latest image). This sucks since that part of the if isn't even executed, but I guess shells aren't infinitely forgiving when parsing

@eigood
Copy link

eigood commented Jan 17, 2017

I'd first attempt to use /dev/tcp(or /dev/udp) in a redirect first, before attempting external programs. There's the issue with not being able to set the timeout when doing it this way, however, that can be worked around by using a background process that sends a signal to the parent.

@jacobsvante
Copy link

/dev/tcp sounds like an interesting approach. @eigood Have you seen an implementation of this? Would be especially interesting to see HTTP status code parsing.

@DukeyToo DukeyToo closed this by deleting the head repository Oct 22, 2023
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.

Support running on OSX
5 participants