Change in ovirt-engine[master]: userportal, webadmin: IE9 header value

awels at redhat.com awels at redhat.com
Fri Aug 8 19:40:30 UTC 2014


Alexander Wels has submitted this change and it was merged.

Change subject: userportal,webadmin: IE9 header value
......................................................................


userportal,webadmin: IE9 header value

- Fixed issue where IE9 was reporting missing headers as
  blank instead of null. This would cause initialization
  routined to be executed, and that would cause another
  request which would cause the header to be missing again
  and one would end up in an endless loop of requests. The
  patch checks for both null and blank so the loop never
  happens.

Change-Id: I4144a585a1dec1e736d30f7b7b698687543f7936
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1114863
Signed-off-by: Alexander Wels <awels at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/gin/BaseSystemModule.java
A frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/utils/HttpUtils.java
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/plugin/restapi/RestApiSessionManager.java
3 files changed, 25 insertions(+), 2 deletions(-)

Approvals:
  Alexander Wels: Verified
  Vojtech Szocs: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4144a585a1dec1e736d30f7b7b698687543f7936
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Einav Cohen <ecohen at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list