hookdeck-cli icon indicating copy to clipboard operation
hookdeck-cli copied to clipboard

Global config flag ignored

Open rasmusson opened this issue 1 year ago • 7 comments

I using hookdeck cli with docker and Im trying to run it as another user than root. To accomplish that Im trying to use --config flag of hookdeck cli. No matter what I try, the flag seems to be ignored.

I put the config file in /tmp/.config/hookdeck/config.toml and try to start hookdeck cli with hookdeck listen homeassistant-service:8080 automate --config /tmp/.config/hookdeck/config.toml

I always get "Not connected with any account. Creating a guest account..."

rasmusson avatar Jul 11 '24 15:07 rasmusson

@rasmusson can you share the config you have just so I understand how you're using it? Please remember to remove the secrets before sharing!

alexluong avatar Jul 11 '24 15:07 alexluong

[default] api_key = "KEY" client_id = "cli_iL4DiP3O2SAcG4" device_name = "server" display_name = "Stefan Rasmusson" team_id = "tm_nekwmmmU0Nni" team_mode = "inbound" team_name = "home_automation"

rasmusson avatar Jul 11 '24 17:07 rasmusson

Thanks @rasmusson. Can you also share your Hookdeck CLI version you're running?

Have you tried hookdeck whoami to see which profile you're using?

It seems you may be running the CLI inside Docker? Let me test this out and see what's going on here.

alexluong avatar Jul 11 '24 17:07 alexluong

version 0.8.6 Profile is default

rasmusson avatar Jul 11 '24 17:07 rasmusson

Hi @rasmusson, thanks for sharing. I tried recreating your issue locally but unable to encounter this problem. I noticed a few things, so maybe you can help explore these ideas maybe?

1: The config you shared seems a little dated. It looks like it's a "CI" client where you ran hookdeck ci to generate this key. Not a problem, but the schema seems like an old version. I would highly encourage you to re-run hookdeck ci or hookdeck login to get a new set of credentials. I think this should solve your issue.

2: Because you used /tmp/... so I assume you're working within a Docker container. Can you try recreating this issue locally first to make sure the issue is not Docker-specific?

3: You're using v0.8.6, which is a bit older version. We've had 2 breaking changes since. This should NOT be an issue here since I tried using this version too and things still work on my end, but maybe you should consider updating to the latest version, just to be up to date?

Again, I think the issue is probably the old credentials here. Sorry for the breaking change, we definitely should have considered a better migration path last year when we rolled out a new version where the CLI handles API keys differently.

Please let me know if you still run into issues here and I'd be happy to investigate further!

alexluong avatar Jul 16 '24 09:07 alexluong

@alexluong - just to quickly confirm, you have see the --config flag work as expected?

leggetter avatar Jul 16 '24 15:07 leggetter

@leggetter yes, when I tried the config flag, everything was working as expected. I tested on the latest version and v0.8.6.

alexluong avatar Jul 16 '24 15:07 alexluong