tailscale / docker-extension

Docker Desktop extension adding Tailscale networking.
61 stars 12 forks source link

fix: broken args, up is a separate arg #54

Closed benoitf closed 10 months ago

benoitf commented 11 months ago

while testing this extension in Podman Desktop and the support of Docker Desktop extensions, there was an error reported on my side.

I figured out that it's because the arguments are not separated.

I guess Docker Desktop is sending the call through a CLI call so space don't matters but if we send using REST API, 'tailscale up' should be split into two args, 'tailscale' and 'up'

benoitf commented 10 months ago

@rosszurowski is that this extension is still maintained ? if so I would love to have this fix being merged

benoitf commented 10 months ago

thank you @rosszurowski 🚀