Change in ovirt-engine[master]: packaging: setup: updated early statement for /api
Ofer Schreiber has submitted this change and it was merged.
Change subject: packaging: setup: updated early statement for /api
......................................................................
packaging: setup: updated early statement for /api
The current proxy configuration removes Expect
headers incorrectly:
1) The 'early' statement removed by the commit
4acbcb524 is still required.
2) According to mod_headers documentation at [1],
the directive cannot be used as nested in <Location>:
"Because early directives are processed before the request
path's configuration is traversed, early headers can only
be set in a main server or virtual host context.
Early directives cannot depend on a request path, so they
will fail in contexts such as <Directory> or <Location>"
The new implementation brings back the 'early' directive and
relocates the statement into main context.
[1] http://httpd.apache.org/docs/current/mod/mod_headers.html
Change-Id: I4dda234b7669dcd6c6eb76728a8536ba49247393
Bug-Url: https://bugzilla.redhat.com/849442
Related-To: https://bugzilla.redhat.com/979737
Signed-off-by: Alex Lourie <alourie(a)redhat.com>
---
M packaging/conf/ovirt-engine-proxy.conf.v2.in
1 file changed, 16 insertions(+), 8 deletions(-)
Approvals:
Ofer Schreiber: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/16882
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I4dda234b7669dcd6c6eb76728a8536ba49247393
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alex Lourie <alourie(a)redhat.com>
Gerrit-Reviewer: Alex Lourie <alourie(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpastern(a)redhat.com>
Gerrit-Reviewer: Moran Goldboim <mgoldboi(a)redhat.com>
Gerrit-Reviewer: Ofer Schreiber <oschreib(a)redhat.com>
Gerrit-Reviewer: Sandro Bonazzola <sbonazzo(a)redhat.com>