Michael Pasternak has posted comments on this change.

Change subject: packaging: api: redirect /api to /ovirt-engine/api using 
rewrite rules
......................................................................


Patch Set 1:

(1 comment)

....................................................
File backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/web.xml
Line 26:   <session-config>
Line 27:     <session-timeout>180</session-timeout>
Line 28:     <cookie-config>
Line 29:       <path>/</path>
Line 30:     </cookie-config>
no, it was true only in redirectservlet, the change you did in this patch is to 
modify
url before it got processed, so user actually never works with /api,

/api/events is wrapped to the /ovirt-engine/api/events before server process it,
same will be true for any other request that has /api in root,

this is why HttpSession is always available, cause only one context is used - 
/ovirt-engine/api
Line 31:   </session-config>


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2ad9bb362719c7805be9f69d7e14bd3ad7b1f5c7
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to