-
Notifications
You must be signed in to change notification settings - Fork 7.7k
Closed
Labels
area/authenticationIndicates an issue on Authentication areaIndicates an issue on Authentication areaarea/corekind/enhancementCategorizes a PR related to an enhancementCategorizes a PR related to an enhancementrelease/26.3.0team/core-clientsteam/core-shared
Description
Description
Hi, guys!
I found some minor bugs in the code while running SAST. There are some signs of bad copy/paste.
In particular, the code was changed in Commit 2c238b9 and this part of code is still in the main branch.
Line 85 in 3099570
logger.infof("Too many sessions related to the current client for this user. Session count: %s", userSessionCountForRealm); |
It looks like userSessionCountForClient should be used instead of userSessionCountForRealm.
This code is in a block that compares userSessionCountForClient to determine if userClientLimit has been exceeded, so it would make sense to output the value of userSessionCountForClient to the logs.
Discussion
No response
Motivation
No response
Details
No response
Found by Linux Verification Center with SVACE
Metadata
Metadata
Assignees
Labels
area/authenticationIndicates an issue on Authentication areaIndicates an issue on Authentication areaarea/corekind/enhancementCategorizes a PR related to an enhancementCategorizes a PR related to an enhancementrelease/26.3.0team/core-clientsteam/core-shared