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

Backport of UI: better calculation for advanced secret in KV v2 into release/1.15.x #24525

Conversation

hc-github-team-secure-vault-core
Copy link
Collaborator

Backport

This PR is auto-generated from #24513 to be assessed for backporting due to the inclusion of the label backport/1.15.x.

The below text is copied from the body of the original PR.


In #24308 we inadvertently introduced an issue where secrets with values that include { would be locked to the JSON view. This PR makes the check in a more intelligent way so that the original intended behavior is preserved, but secret values are not shown.

Now, string values with { are shown as inputs instead of with the JSON editor:
Screenshot 2023-12-13 at 1 56 28 PM
Screenshot 2023-12-13 at 1 56 33 PM

Out of scope for this backport but as a future improvement, we will add obscurity to the JSON view that is toggleable


Overview of commits

@hc-github-team-secure-vault-core hc-github-team-secure-vault-core force-pushed the backport/24504-enhancement-for-kv-engine-ui-behavior/preferably-just-dane branch from 066fcca to e653cc8 Compare December 13, 2023 21:38
@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label Dec 13, 2023
@hashishaw hashishaw added this to the 1.15.5 milestone Dec 13, 2023
@hashishaw hashishaw enabled auto-merge (squash) December 13, 2023 21:40
@hashishaw hashishaw merged commit 4ae8636 into release/1.15.x Dec 13, 2023
@hashishaw hashishaw deleted the backport/24504-enhancement-for-kv-engine-ui-behavior/preferably-just-dane branch December 13, 2023 21:51
Copy link

Build Results:
All builds succeeded! ✅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed ui
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants