Change in ovirt-engine[master]: utils: RedirectServlet: replace localhost to engine host

oourfali at redhat.com oourfali at redhat.com
Tue Jan 13 17:42:40 UTC 2015


Oved Ourfali has submitted this change and it was merged.

Change subject: utils: RedirectServlet: replace localhost to engine host
......................................................................


utils: RedirectServlet: replace localhost to engine host

as the redirect happens at client side, the client needs explicit host
name, while localhost may appear at engine configuration in regard of
where resource may be.

to overcome this, replace localhost with the Host header or if not
available with known engine name.

Change-Id: I155489de230124339be4f2493686c83ebaee3a68
Signed-off-by: Alon Bar-Lev <alonbl at redhat.com>
---
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/servlet/RedirectServlet.java
1 file changed, 66 insertions(+), 9 deletions(-)

Approvals:
  Alon Bar-Lev: Verified
  Ravi Nori: Looks good to me, but someone else must approve
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I155489de230124339be4f2493686c83ebaee3a68
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Shirly Radco <sradco at redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <didi at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list