Alexander Wels has posted comments on this change. Change subject: services: add proxy servlet ......................................................................
Patch Set 5: (1 comment) http://gerrit.ovirt.org/#/c/30740/5/backend/manager/modules/uutils/src/main/java/org/ovirt/engine/core/uutils/servlet/ProxyServletBase.java File backend/manager/modules/uutils/src/main/java/org/ovirt/engine/core/uutils/servlet/ProxyServletBase.java: Line 127: protected void doGet(HttpServletRequest request, HttpServletResponse response) Line 128: throws ServletException, IOException { Line 129: Line 130: /* Line 131: * short circute if we have nothing to do. typo, should be 'short circuit' Line 132: */ Line 133: if (url == null) { Line 134: response.sendError(response.SC_NOT_FOUND, "Cannot proxy, no URL is configured."); Line 135: return; -- To view, visit http://gerrit.ovirt.org/30740 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ia26768ec7a0e06a20f657cae40f6a51303970b44 Gerrit-PatchSet: 5 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Alexander Wels <[email protected]> Gerrit-Reviewer: Alon Bar-Lev <[email protected]> Gerrit-Reviewer: Oved Ourfali <[email protected]> Gerrit-Reviewer: Ravi Nori <[email protected]> Gerrit-Reviewer: Yaniv Dary <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
