Skip to content

Conversation

@smolnar82
Copy link
Contributor

What changes were proposed in this pull request?

The Token Integration link on the Home page must be hidden if the required knox.token.hash.key Gateway-level alias is missing.

How was this patch tested?

Manual testing.

Without the alias:
Screenshot 2021-10-31 at 21 09 54

With the alias (running bin/knoxcli.sh generate-jwk --saveAlias knox.token.hash.key):

Screenshot 2021-10-31 at 21 29 50

@lmccay
Copy link
Contributor

lmccay commented Oct 31, 2021

LGTM!

+1

Copy link
Contributor

@lmccay lmccay left a comment

Choose a reason for hiding this comment

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

+1

@smolnar82 smolnar82 merged commit 86c1a96 into apache:master Oct 31, 2021
@smolnar82 smolnar82 deleted the KNOX-2685 branch October 31, 2021 21:16
smolnar82 added a commit that referenced this pull request Oct 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants