Change in ovirt-engine[master]: aaa: Consolidate SSO session validation requests

mperina at redhat.com mperina at redhat.com
Tue Oct 4 07:28:13 UTC 2016


Martin Peřina has submitted this change and it was merged.

Change subject: aaa: Consolidate SSO session validation requests
......................................................................


aaa: Consolidate SSO session validation requests

Engine makes multiple queries to SSO
to validate each exisitng session. This
could cause performance issue when there
are thousands of sessions. Combining all
requests into one request to fetch all
the statuses in one request can resolve
the performance issue.

When SSO clears an engine session associated
with a token by invoking TerminateSessionsForToken
SSO_ACCESS_TOKEN_PARAMETER value in the
map is null. This can lead to problems when
equality is checked with the contents of
the map for the token.

Change-Id: Ie63dd4e85ac4e4b25cd2884d07d84483dc8add17
Bug-Url: https://bugzilla.redhat.com/1379805
Signed-off-by: Ravi Nori <rnori at redhat.com>
---
M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/SsoOAuthServiceUtils.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/SessionDataContainer.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/aaa/SessionDataContainerTest.java
M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/search/DirectorySearch.java
M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/utils/SsoConstants.java
5 files changed, 58 insertions(+), 24 deletions(-)

Approvals:
  Martin Peřina: Looks good to me, approved
  Ravi Nori: Verified
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/64937
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ie63dd4e85ac4e4b25cd2884d07d84483dc8add17
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list