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 <alo...@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 <alo...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: Ravi Nori <rn...@redhat.com>
Gerrit-Reviewer: Shirly Radco <sra...@redhat.com>
Gerrit-Reviewer: Yedidyah Bar David <d...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to