logging 'flushed' when we get a response from the endpoint and using …

…hostname rather than host when we create the endpoint client
jlertle · Jul 7, 2010 · 9d4e09b · 9d4e09b
1 parent 7ed32a5
commit 9d4e09b
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/lib/pixel-ping.js b/lib/pixel-ping.js
@@ -38,7 +38,9 @@
     request = endpoint.request('POST', endParams.pathname, endHeaders);
     request.write(data);
     request.end();
-    return console.info('--- flushed ---');
+    return request.on('response', function(response) {
+      return console.info('--- flushed ---');
+    });
   };
   log = function(hash) {
     var _b, _c, hits, key;
@@ -76,7 +78,7 @@
   if (config.endpoint) {
     console.info("Flushing hits to " + config.endpoint);
     endParams = url.parse(config.endpoint);
-    endpoint = http.createClient(endParams.port || 80, endParams.host);
+    endpoint = http.createClient(endParams.port || 80, endParams.hostname);
     endHeaders = {
       host: endParams.host,
       'Content-Type': 'application/x-www-form-urlencoded'

diff --git a/src/pixel-ping.coffee b/src/pixel-ping.coffee
@@ -32,7 +32,8 @@ flush: ->
   request: endpoint.request 'POST', endParams.pathname, endHeaders
   request.write data
   request.end()
-  console.info '--- flushed ---'
+  request.on 'response', (response) ->
+    console.info '--- flushed ---'
 
 # Log the contents of the hits to stdout.
 log: (hash) ->
@@ -61,7 +62,7 @@ pixel:        fs.readFileSync(__dirname + '/pixel.gif')
 if config.endpoint
   console.info "Flushing hits to $config.endpoint"
   endParams:   url.parse config.endpoint
-  endpoint:    http.createClient endParams.port || 80, endParams.host
+  endpoint:    http.createClient endParams.port || 80, endParams.hostname
   endHeaders:  {host : endParams.host, 'Content-Type': 'application/x-www-form-urlencoded'}
 
 # Don't let exceptions kill the server.