Change in ovirt-engine[master]: aaa: Honor 'Session-TTL' header

oliel at redhat.com oliel at redhat.com
Wed Jul 8 13:45:33 UTC 2015


Ori Liel has submitted this change and it was merged.

Change subject: aaa: Honor 'Session-TTL' header
......................................................................


aaa: Honor 'Session-TTL' header

Currently API requests may have "Session-TTL=xxx" header on them.
This should set the 'soft-limit' (in minutes) for this session in
the engine, but in fact is ignored.

This patch causes this header to be honored. For requests that open
a new session (meaning: cause a log-in), the value of the header is
saved and used later every time the session is refreshed (instead of
the default ConfigValues.UserSessionTimeOutInterval)

Bug-Url: http://bugzilla.redhat.com/1157751
Change-Id: If6d01d1afc2045ee1ed267512e8d5755a6413d4f
Signed-off-by: Ori Liel <oliel at redhat.com>
---
M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/filters/FiltersHelper.java
M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/filters/LoginFilter.java
M backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/filters/RestApiSessionMgmtFilter.java
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/SessionDataContainer.java
A backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/SetSesssionSoftLimitCommand.java
A backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/SetSesssionSoftLimitCommandParameters.java
M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/VdcActionType.java
7 files changed, 144 insertions(+), 36 deletions(-)

Approvals:
  Ori Liel: Verified
  Jenkins CI: Passed CI tests
  Moti Asayag: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: If6d01d1afc2045ee1ed267512e8d5755a6413d4f
Gerrit-PatchSet: 12
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Ori Liel <oliel at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list