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.

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

feat(profile/token): Allow user to specify how long token must be valid. #1340

Merged
merged 3 commits into from
Nov 11, 2024

Conversation

kpfleming
Copy link
Contributor

@kpfleming kpfleming commented Nov 8, 2024

The default token validity period for 'fastly profile token' is 5 minutes, but users may want to ensure that the token is valid for a longer period (or only a shorter period). This PR adds 'fastly profile token --ttl=<period>', allowing the user to specify the validity period in seconds, minutes, or hours (or some combination).

The default token validity period for 'fastly profile token' is 5
minutes, but users may want to ensure that the token is valid for a
longer period (or only a shorter period). This PR adds 'fastly profile
token --ttl=<period>', allowing the user to specify the validity
period in seconds, minutes, or hours (or some combination).
@kpfleming kpfleming marked this pull request as ready for review November 8, 2024 20:29
Profiles with non-OIDC tokens (tokens generated using the traditional
Fastly token-creation methods) do not contain expiration information,
so the validity-checking in the `fastly profile token` command should
not attempt to validate them.
The `fastly profile update` command is not the best suggestion in this
situation; the user should directly invoke `fastly sso` instead.
Copy link
Member

@philippschulte philippschulte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, thank you!

@kpfleming kpfleming merged commit 91a6e3f into main Nov 11, 2024
9 checks passed
@kpfleming kpfleming deleted the cdtool-960 branch November 11, 2024 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants