Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

print tenant context on output [CLI-35] #104

Closed
woloski opened this issue Feb 22, 2021 · 4 comments
Closed

print tenant context on output [CLI-35] #104

woloski opened this issue Feb 22, 2021 · 4 comments
Labels
enhancement An enhancement or improvement to the SDK that could not be otherwise categorized as a new feature

Comments

@woloski
Copy link

woloski commented Feb 22, 2021 •
edited
Loading

When running a command I would like to know which tenant I am running this into.

For eg. I used login to setup a tenant (I forgot I had one already configured). Then run apps list and got "token expired".

Loading apps... failed
 ▸    401 Unauthorized: Expired token received for JSON Web Token validation

Checked config.json and realized I had an old tenant configure whose token was expired already (btw we need to implement Refresh Token... will create another issue for that). In any case, if we include the tenant context in each run I would know which tenant I am running against.
@Widcket
Copy link
Contributor

Widcket commented Feb 22, 2021

we need to implement Refresh Token

There's a card for that already.

@Widcket
Copy link
Contributor

Widcket commented Feb 22, 2021

We already include the tenant we're running against (on the success case). We'd need to show it as well when the command fails. I'll add a card for this.

@Widcket Widcket added the enhancement An enhancement or improvement to the SDK that could not be otherwise categorized as a new feature label Feb 22, 2021
@Widcket
Copy link
Contributor

Widcket commented Feb 22, 2021

Created card CLI-35 to track this. Thanks for raising this.

@Widcket Widcket changed the title print tenant context on output print tenant context on output [CLI-35] Feb 22, 2021
@Widcket
Copy link
Contributor

Widcket commented Feb 27, 2021

Done in #107

@Widcket Widcket closed this as completed Feb 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement An enhancement or improvement to the SDK that could not be otherwise categorized as a new feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@woloski @Widcket