-
Notifications
You must be signed in to change notification settings - Fork 257
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.
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
Create API tokens with team slug #980
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple of suggestions for the help.
command/api/token/create.go
Outdated
@@ -21,7 +21,7 @@ func createCommand() cli.Command { | |||
Name: "create", | |||
Action: cli.ActionFunc(createAction), | |||
Usage: "create a new token", | |||
UsageText: `**step api token create** <team-id> <crt-file> <key-file> | |||
UsageText: `**step api token create** <team-id>|<team-slug> <crt-file> <key-file> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would use just and use the text in the positional arguments to actually describe what that argument means.
UsageText: `**step api token create** <team-id>|<team-slug> <crt-file> <key-file> | |
UsageText: `**step api token create** <team> <crt-file> <key-file> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
command/api/token/create.go
Outdated
@@ -32,8 +32,8 @@ func createCommand() cli.Command { | |||
|
|||
## POSITIONAL ARGUMENTS | |||
|
|||
<team-id> | |||
: UUID of the team the API token will be issued for. This is available in the Smallstep dashboard. | |||
<team-id>|<team-slug> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
<team-id>|<team-slug> | |
<team> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Name of feature:
API auth tokens with team slug
Pain or issue this feature alleviates:
Previous you needed to supply the team UUID to get an auth token for the API.
Why is this important to the project (if not answered above):
Is there documentation on how to use this feature? If so, where?
In what environments or workflows is this feature supported?
Getting an auth token from the Gateway API.
In what environments or workflows is this feature explicitly NOT supported (if any)?
Supporting links/other PRs/issues:
💔Thank you!