Conversation
Without this when cloudflared is started with Mac OS launch agent it errors with:
```
Use `cloudflared tunnel run` to start tunnel {UUID}
Use `cloudflared tunnel run` to start tunnel {UUID}
Use `cloudflared tunnel run` to start tunnel {UUID}
```
This patch is suggested by cloudflare#327 (comment)
| } else { | ||
| // empty extra args | ||
| return make([]string, 0), nil | ||
| return []string{ |
There was a problem hiding this comment.
Instead of limiting this to only be able to run with tunnel parameters, could we improve this to allow any tunnel sub-commands to be passed in?
For example, if someone wanted to run cloudflared access tcp as a service, we should allow them to also setup using this command similar to the following:
cloudflared service install -- access tcp --hostname test.example.com --url localhost:8080
We would still like to allow a user to provide just their tunnel token, so we can still leave and try and parse that first to insert the tunnel run --token <TOKEN> command, but if that fails, we should just provide whatever the user provides as a subcommand to cloudflared.
I think that solves the problem you are trying to solve here while also expanding the functionality to be expressive for other folks that might try something similar. What do you think?
|
Where did this end up going? I am just trying to get cloudflared to persist my tunnel after reboots, cant believe how painful this is |
cloudflared service installis not working as documented at #327When cloudflared is started with Mac OS launch agent it errors with:
A fix is suggested at #327 (comment)
And acknowledged by https://community.cloudflare.com/t/tunnel-error-on-macos-when-installed-as-service/339543/4
However, collective costs keep compounding.
The fix seems simple at least for my use case where only a single tunnel is used. But Not sure if there's other negative side effects.
cloudflared service installproduces such.plistas opposed to