-
Notifications
You must be signed in to change notification settings - Fork 714
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
Add cloudflared tunnel ready command #1135
base: master
Are you sure you want to change the base?
Conversation
Co-authored-by: Julien Laffaye <jlaffaye@freebsd.org>
@jlaffaye I there anything I can do to fast-track the PR somehow ? |
Am I to understand that the If that is a correct assumption, I would much rather use It at least checks that ready connections are above 0 I've renamed the sub command to |
@firecow That makes sense. 👍🏻 |
A merge of master into your branch may be in order because I was getting an error about "UDP Buffer Sizes" that forced me to merge in master. Which resolved it. But past that this seems to work fine on a quick test. I stood up a separate service at port 8081 to make a cloudflare tunnel with. Then wrote a docker healthcheck into a docker-compose.yml. Which had my local IP there: healthcheck:
test: tunnel --metrics 192.168.0.15:8081 ready || exit 1 Which worked great. The timeout was a little high at 30 seconds but it's fine. |
Master has been merged. |
This would make it possible to have a Docker image without wget and curl, and still make docker healthchecking possible
closes #1126