Login command is broken about clockidup

[maelvls]

Looks like I introduced a "hidden shared state" bug. I used the same http.DefaultClient in two calls: the first call would "wrap" the transport in http.DefaultClient a first time, and the second call would wrap a second time; the most nested one (i.e., the first call to NewClockify) would override the token of the least nested transport...

Ahh if only the use of http.DefaultClient was deprecated... Also, http.DefaultTransport should have some creation function to avoid state sharing...

I fixed the bug in v0.3.0, and you can install it with:

go install github.com/maelvls/clockidup@latest

To check that you have the latest version:

❯ go version -m $(which clockidup)
/home/mvalais/go/bin/clockidup: go1.16.3
        path    github.com/maelvls/clockidup
        mod     github.com/maelvls/clockidup    v0.3.0

NOTE: I changed the name of the config file from ~/.config/standup.yml to ~/.config/clockidup.yml.

Let me know if that works better or not!

[davidcollom]

👍 Confirmed working by re-installing and logging in again.

Thanks for the speedy response! approve the config file name change too makes total sense! 👍

Thanks!