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

UI: Align auth method config ttl value with system setting #26658

Closed
wants to merge 9 commits into from

Conversation

hellobontempo
Copy link
Contributor

If a ttl has not been configured for a mount then the mount uses system defaults which is conveyed by a ttl: 0.
Screenshot 2024-04-25 at 5 31 54 PM

It's unclear what the actual default value is unless you run vault read sys/auth/token/tune.

⇒ vault read sys/auth/token/tune
Key                  Value
---                  -----
default_lease_ttl    768h
description          token based credentials
force_no_cache       false
max_lease_ttl        768h
token_type           default-service

The backend PR #23914 updates the sys/internal/ui/mounts endpoint to return the default TTL value if there is not a mount-specific TTL configured.

To leverage this update in the UI, we have to change the query for those records from sys/auth/:path to also use sys/internal/ui/mounts. By updating this request we can now see the system default TTL value:
Screenshot 2024-04-25 at 5 41 44 PM

@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label Apr 26, 2024
@hellobontempo hellobontempo changed the base branch from main to vault-21112/fix-mountinfo-to-match-tune-read April 26, 2024 00:42
Copy link

CI Results: failed ❌

Base automatically changed from vault-21112/fix-mountinfo-to-match-tune-read to main April 26, 2024 14:53
@hellobontempo
Copy link
Contributor Author

closing in favor of #26663

@hellobontempo hellobontempo deleted the ui/align-auth-method-ttl-with-tune branch May 6, 2024 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants