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

Output the duration used when it is overridden #111

Merged

Conversation

ystros
Copy link
Contributor

@ystros ystros commented Aug 19, 2021

Replaces #110

Newer versions of the CredHub server will include two new fields -
duration_overridden, which indicates the server-configured minimum was
used instead of the requested duration, and duration_used, which
indicates the final duration used to generate / regenerate the
certificate.

When these fields are present, and the duration_overridden flag is true,
the CLI will print out:
duration_overridden_to: X

with the value of the actual duration used.

Co-authored-by: Preethi Varambally pvarambally@pivotal.io
Co-authored-by: Brian Upton bupton@vmware.com
Co-authored-by: Pablo Rodas prodas@vmware.com

Newer versions of the CredHub server will include two new fields -
duration_overridden, which indicates the server-configured minimum was
used instead of the requested duration, and duration_used, which
indicates the final duration used to generate / regenerate the
certificate.

When these fields are present, and the duration_overridden flag is true,
the CLI will print out:
duration_overridden_to: X

with the value of the actual duration used.

Co-authored-by: Preethi Varambally <pvarambally@pivotal.io>
Co-authored-by: Brian Upton <bupton@vmware.com>
Co-authored-by: Pablo Rodas <prodas@vmware.com>
@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this:

https://www.pivotaltracker.com/story/show/179289311

The labels on this github issue will be updated when the story is started.

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.

4 participants