Change in ovirt-engine[master]: aaa: Added usage of AuthRecord.VALID_TO

yzaslavs at redhat.com yzaslavs at redhat.com
Wed Apr 30 10:20:35 UTC 2014


Yair Zaslavsky has submitted this change and it was merged.

Change subject: aaa: Added usage of AuthRecord.VALID_TO
......................................................................


aaa: Added usage of AuthRecord.VALID_TO

The SessionDataContainer cleans sessions after
UserSessionTimeoutInterval minutes (30 minutes).
AuthRecord.VALID_TO may contain a date in the future
that holds the time the session should expire.
If this time is bigger than the time of the next sessions invalidation
interval,
the sessions invalidation of the session data container
will invalid it anyway.
If it is smaller than the next interval, and a login attempt is performed
using this session, and the expiration time has arrived,
the session will be removed from the session data container, and the user
will be notified the session was timed out.
Change-Id: I53e4a371c1bae8d2480ddd2af921a560c6fe9a85
Topic: AAA
Signed-off-by: Yair Zaslavsky <yzaslavs at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/Backend.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/LoginBaseCommand.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/session/SessionDataContainer.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/AbstractQueryTest.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/CommandBaseTest.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/QueriesCommandBaseTest.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/RemoveVdsCommandTest.java
M backend/manager/modules/bll/src/test/java/org/ovirt/engine/core/bll/session/SessionDataContainerTest.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/config/ConfigValues.java
M packaging/dbscripts/upgrade/pre_upgrade/0000_config.sql
10 files changed, 142 insertions(+), 121 deletions(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved



-- 
To view, visit http://gerrit.ovirt.org/26975
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I53e4a371c1bae8d2480ddd2af921a560c6fe9a85
Gerrit-PatchSet: 34
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list