Alon Bar-Lev has posted comments on this change.

Change subject: aaa: add support for basic athentication
......................................................................


Patch Set 7:

(1 comment)

http://gerrit.ovirt.org/#/c/37299/7/backend/manager/modules/enginesso/src/main/webapp/WEB-INF/web.xml
File backend/manager/modules/enginesso/src/main/webapp/WEB-INF/web.xml:

Line 30:     </context-param>
Line 31:     <context-param>
Line 32:         <param-name>enable-basic-auth</param-name>
Line 33:         <param-value>false</param-value>
Line 34:     </context-param>
we should move all these to configuration %{xxx} notation.
Line 35: 
Line 36:     <!-- Branding Servlet -->
Line 37:     <servlet>
Line 38:         <servlet-name>BrandingServlet</servlet-name>


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

Gerrit-MessageType: comment
Gerrit-Change-Id: If09285f0e6cd8909f21aa7e88ae1a3c1a30763c2
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
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