Skip to content

KNOX-2618 - INFO level logging added about removing expired tokens from the DB#455

Merged
smolnar82 merged 1 commit intoapache:masterfrom
smolnar82:KNOX-2618
Jun 8, 2021
Merged

KNOX-2618 - INFO level logging added about removing expired tokens from the DB#455
smolnar82 merged 1 commit intoapache:masterfrom
smolnar82:KNOX-2618

Conversation

@smolnar82
Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Instead of logging the number of expired tokens removed from the DB at DEBUG level, Knox logs the

How was this patch tested?

Tested manually:

2021-06-08 12:45:54,837 INFO  service.knoxtoken (TokenResource.java:getAuthenticationToken(570)) - Knox Token service (homepage) issued token eyJqa3...sHsZgg (ab56217a...aa2afedee2bc)
2021-06-08 12:45:56,445 INFO  service.knoxtoken (TokenResource.java:getAuthenticationToken(570)) - Knox Token service (homepage) issued token eyJqa3...qC7Kag (99212174...e50a90ccffa5)
2021-06-08 12:45:57,793 INFO  service.knoxtoken (TokenResource.java:getAuthenticationToken(570)) - Knox Token service (homepage) issued token eyJqa3...WG117A (9bf33980...be4eb1ac08dc)
2021-06-08 12:45:58,925 INFO  service.knoxtoken (TokenResource.java:getAuthenticationToken(570)) - Knox Token service (homepage) issued token eyJqa3...RC_BwQ (895a34a8...69d0574a7324)
2021-06-08 12:45:59,887 INFO  service.knoxtoken (TokenResource.java:getAuthenticationToken(570)) - Knox Token service (homepage) issued token eyJqa3...qZUNtw (65bf1dfa...b52853a1053f)
2021-06-08 12:46:00,754 INFO  service.knoxtoken (TokenResource.java:getAuthenticationToken(570)) - Knox Token service (homepage) issued token eyJqa3...RnkKRA (3a7824b7...a37942c02ce9)
...
2021-06-08 12:47:09,483 INFO  token.state (JDBCTokenStateService.java:evictExpiredTokens(218)) - Removing 6 expired token(s) from the database: 895a34a8...69d0574a7324, ab56217a...aa2afedee2bc, 3a7824b7...a37942c02ce9, 65bf1dfa...b52853a1053f, 99212174...e50a90ccffa5, 9bf33980...be4eb1ac08dc
2021-06-08 12:47:09,491 INFO  token.state (DefaultTokenStateService.java:getExpiredTokens(372)) - Evicting expired token 3a7824b7...a37942c02ce9
2021-06-08 12:47:09,492 INFO  token.state (DefaultTokenStateService.java:getExpiredTokens(372)) - Evicting expired token 895a34a8...69d0574a7324
2021-06-08 12:47:09,492 INFO  token.state (DefaultTokenStateService.java:getExpiredTokens(372)) - Evicting expired token ab56217a...aa2afedee2bc
2021-06-08 12:47:09,492 INFO  token.state (DefaultTokenStateService.java:getExpiredTokens(372)) - Evicting expired token 99212174...e50a90ccffa5
2021-06-08 12:47:09,492 INFO  token.state (DefaultTokenStateService.java:getExpiredTokens(372)) - Evicting expired token 9bf33980...be4eb1ac08dc
2021-06-08 12:47:09,492 INFO  token.state (DefaultTokenStateService.java:getExpiredTokens(372)) - Evicting expired token 65bf1dfa...b52853a1053f

@smolnar82 smolnar82 requested review from lmccay and pzampino June 8, 2021 10:51
@smolnar82 smolnar82 self-assigned this Jun 8, 2021
@smolnar82 smolnar82 merged commit 59c3a3d into apache:master Jun 8, 2021
@smolnar82 smolnar82 deleted the KNOX-2618 branch June 8, 2021 13:04
stoty pushed a commit to stoty/knox that referenced this pull request May 14, 2024
…d tokens from the DB (apache#455)

Change-Id: I3fffc84e475bde02a4b75a896d502085027b2955
stoty pushed a commit to stoty/knox that referenced this pull request May 14, 2024
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.

1 participant