Yair Zaslavsky has posted comments on this change.

Change subject: AAA: Introduce usage of Acct
......................................................................


Patch Set 3:

Hi,
About Session expeiration - maybe i wasn't clear.
What's wrong with the suggestion to implement it at SessionDataContainer?
What I meant about my discussion with Vojtech is that he said that if webadmin 
tries to access with session x, and session x does not exist at the session 
data container, the web admin will present that the user was logged out due to 
session timeout.
however, the logic of expiring the session exists at the server side.
in addition, i have checked all our web.xml file, and only rest-api has a 
definition of session-timeout.
Do you still think we need an interceptor here?

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ief13d233d11b7ab32b328735b4f58ec7cffff567
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to