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.

Sign up for GitHub

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

Jump to bottom

KNOX-2966 - Improved logging around KnoxSSO cookie management #802

Merged
merged 1 commit into from
Oct 10, 2023
Merged

KNOX-2966 - Improved logging around KnoxSSO cookie management #802

merged 1 commit into from
Oct 10, 2023

Conversation

smolnar82
Copy link
Contributor

What changes were proposed in this pull request?

Better logging experience in KnoxSSO Cookie management.

How was this patch tested?

Deployed Knox, logged in to the Knox Home page and checked logs:

2023-10-10 13:03:51,102 f268dade-b9ba-425d-b0d6-d3e97d6b9243 INFO  knox.gateway (CookieUtils.java:getCookiesForName(46)) - Unable to find cookie with name: hadoop-jwt
...
2023-10-10 13:04:01,683 d4e582d3-8b08-46cd-8540-60315b08a819 DEBUG service.knoxsso (WebSSOResource.java:addJWTHadoopCookie(402)) - Adding the following JWT token as a cookie: eyJraW...VHcD2Q
2023-10-10 13:04:01,684 d4e582d3-8b08-46cd-8540-60315b08a819 INFO  service.knoxsso (WebSSOResource.java:addJWTHadoopCookie(428)) - JWT cookie eyJraW...VHcD2Q successfully added.

@smolnar82 smolnar82 self-assigned this Oct 10, 2023
@smolnar82 smolnar82 merged commit 895022c into apache:master Oct 10, 2023
2 checks passed
@smolnar82 smolnar82 deleted the KNOX-2966 branch October 10, 2023 12:26
stoty pushed a commit to stoty/knox that referenced this pull request May 14, 2024
@smolnar82
CDPD-62366, KNOX-2966: Improved logging around KnoxSSO cookie managem…
b1488e8 
…ent (apache#802)

Change-Id: I5969fa76c3fda04fb6c24c1aeeec14025dc9362c
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zeroflag zeroflag Awaiting requested review from zeroflag

Assignees

@smolnar82 smolnar82

Labels
logging
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@smolnar82