rs-cli: WIP on auth, switch to anyhow #75

Merged

targeting ben/cli-auth from huy55465/auto-quvvmlqlomom

Merged opened by and assigned to

huy55465.tngl.sh 1d ago targeting ben/cli-auth from huy55465/auto-quvvmlqlomom

Commits 7

auth,api,top: add API w/ swagger docs

+610 -294

openidconnect: disable feature lint

+2 -1

treewide: format

+7 -5

rs-cli: auth allow specify token path

+21 -10

rs-cli: wip on saving token to data local

+86 -18

rs-cli: switch to anyhow

+45 -54

rs-cli: add auth command

+112

Comments

v0 1d ago

pkgs/rs-cli/cli.rs:57 Maybe make a TODO to set a default here, once we know hostnames and a few more details.

pkgs/rs-cli/main.rs:20 You can do anyhow::Result here 🤓

Did you run clippy on this? (cargo clippy -- -D warnings --no-deps) it will complain about unused trait imports like anyhow::Context

pkgs/rs-cli/auth.rs:15 anyhow::Context::with_context would be preferred here so it doesn't make the string always

v1 1d ago

whoops, I'm dumb, that didn't compile

v2 1d ago

Otherwise LGTM so far

v3 20h ago

v4 20h ago

That took a lot of rounds...

v5 20h ago

#LGTM