From 948db2fbd0260e5d56ed5ba58df0f5b6599bbe38 Mon Sep 17 00:00:00 2001
From: Christian Fritz <christian.fritz@gmail.com>
Date: Mon, 27 Jun 2016 21:49:12 -0700
Subject: [PATCH] Send content-length header in method responses

- unlike method calls (client), method responses (server) did not yet
  send content-length headers. This was causing problem with at least
  one client (ros's turtlesim, in the context of rosnodejs: publishing
  to /turtlt1/cmd_vel had no effect on the turtle). This commit fixes
  that.
---
 lib/server.js | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/lib/server.js b/lib/server.js
index 37c4c99..e05c5ce 100644
--- a/lib/server.js
+++ b/lib/server.js
@@ -47,7 +47,10 @@ function Server(options, isSecure, onListening) {
           else {
             xml = Serializer.serializeMethodResponse(value)
           }
-          response.writeHead(200, {'Content-Type': 'text/xml'})
+          response.writeHead(200, {
+            'Content-Type': 'text/xml',
+            'Content-Length': Buffer.byteLength(xml, 'utf8')
+          })
           response.end(xml)
         })
       }