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/gh 26599/hide version if redacted #26614

Closed
wants to merge 3 commits into from

Conversation

andaley
Copy link
Contributor

@andaley andaley commented Apr 24, 2024

No description provided.

@github-actions github-actions bot added the hashicorp-contributed-pr If the PR is HashiCorp (i.e. not-community) contributed label Apr 24, 2024
// Fetch seal status with token to get version
const response = yield this.store.adapterFor('cluster').sealStatus(false);
// Fetch seal status to get version
const response = yield this.store.adapterFor('cluster').sealStatus();
Copy link
Contributor Author

Choose a reason for hiding this comment

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

👉 this is the fix -- this is what ensures the UI hides the version by calling /seal-status/ unauthenticated.

@chelshaw i saw that this change was recent and intentionally added in #25998, so i wanted to check in with you about this as a solution. because we're just trying to grab the version property from the response, i wondered if calling the endpoint unauthenticated in this specific instance would be okay? it seems like if we needed more information that's only returned when the call is authenticated that we could just make a separate call then. what do you think?

Copy link

CI Results: failed ❌

@andaley
Copy link
Contributor Author

andaley commented Apr 24, 2024

closing because we intentionally want to show the version if the user is authenticated. we only want to hide the version if the user in unauthenticated.

@andaley andaley closed this Apr 24, 2024
@andaley andaley deleted the ui/gh-26599/hide-version-if-redacted branch April 24, 2024 16:39
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.

1 participant