Saggi Mizrahi has posted comments on this change. Change subject: json-rpc: Protocol detection ......................................................................
Patch Set 14: Code-Review-1 (2 comments) http://gerrit.ovirt.org/#/c/26300/14/vdsm/protoDetector.py File vdsm/protoDetector.py: Line 1: # Sorry if that wasn't clear but also change the name of the module. Line 177: def __init__(self, xmlBinding): Line 178: self.xmlBinding = xmlBinding Line 179: Line 180: def detect(self, data): Line 181: return data.startswith("POST") or data.startswith("PUT") > Just add detection for all HTTP commands and add pass it to an HTTP router Just to clarify that you should do it in a different patch. As what Nir is talking about is not yet in VDSM and this patch is already big enough and adding an HTTP router will just make it bigger. Line 182: Line 183: def handleSocket(self, client_socket, socket_address): Line 184: self.xmlBinding.add_socket(client_socket, socket_address) Line 185: self.log.debug("xml over http detected from %s", socket_address) -- To view, visit http://gerrit.ovirt.org/26300 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Id739a40e2b37dcc175137ec91cd5ec166ad24a75 Gerrit-PatchSet: 14 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Barak Azulay <bazu...@redhat.com> Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Saggi Mizrahi <smizr...@redhat.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: Yeela Kaplan <ykap...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list vdsm-patches@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches