[api] Remove winston logging in favor of custom events

Author: indexzero

Diff for: CHANGELOG.md

@@ -1,5 +1,9 @@
 ## ChangeLog for: node-http-proxy
 
+## Version 0.5.0 - 4/15/2011
+- Remove winston in favor of custom events                 (indexzero)
+
+
 ## Version 0.4.1 - 3/20/2011
 - Include missing dependency in package.json                                  (indexzero)
 

Diff for: lib/node-http-proxy.js

@@ -27,7 +27,6 @@
 var util = require('util'),
     http = require('http'),
     events = require('events'),
-    winston = require('winston'),
     ProxyTable = require('./proxy-table').ProxyTable,
     maxSockets = 100;
 
@@ -105,19 +104,17 @@ exports.createServer = function () {
 
   proxy = new HttpProxy(options);
   server = http.createServer(function (req, res) {
-    winston.verbose('Incoming HTTP request to: ' + req.headers.host + req.url);
+    proxy.emit('request', req, req.headers.host, req.url);
 
     // If we were passed a callback to process the request
     // or response in some way, then call it.
     if (callback) {
       callback(req, res, proxy);
     } 
     else if (port && host) {
-      winston.verbose('Proxying HTTP request to: ' + host + ':' + port);
       proxy.proxyRequest(req, res, port, host);
     }
     else if (proxy.proxyTable) {
-      winston.verbose('Proxying request using proxy table');
       proxy.proxyRequest(req, res);
     }
     else {
@@ -279,12 +276,17 @@ HttpProxy.prototype.proxyRequest = function (req, res, port, host, buffer) {
     host = location.host;
   }
 
+  //
+  // Emit the `start` event indicating that we have begun the proxy operation.
+  //
+  this.emit('start', req, res, host, port);
+  
   //
   // If forwarding is enabled for this instance, foward proxy the
   // specified request to the address provided in `this.options.forward`
   //
   if (this.options.forward) {
-    winston.verbose('Forwarding HTTP request to: ' + this.options.forward.host + ':' + this.options.forward.port);
+    this.emit('forward', req, res, this.options.forward.host, this.options.forward.port);
     this._forwardRequest(req);
   }
 

Diff for: lib/proxy-table.js

@@ -26,8 +26,7 @@
 
 var util = require('util'),
     events = require('events'),
-    fs = require('fs'),
-    winston = require('winston');
+    fs = require('fs');
 
 //
 // ### function ProxyTable (router, silent) 
@@ -119,8 +118,6 @@ ProxyTable.prototype.getProxyLocation = function (req) {
           host = location[0],
           port = location.length === 1 ? 80 : location[1];
 
-      winston.verbose('Proxy Table proxying request to: ' + host + ':' + port);
-      
       return {
         port: port,
         host: host
@@ -136,8 +133,6 @@ ProxyTable.prototype.getProxyLocation = function (req) {
             host = location[0],
             port = location.length === 1 ? 80 : location[1];
 
-        winston.verbose('Proxy Table proxying request to: ' + host + ':' + port);
-
         return {
           port: port,
           host: host

Diff for: package.json

@@ -17,8 +17,7 @@
     "colors": ">= 0.3.0",
     "optimist": ">= 0.1.6",
     "request": ">= 1.9.0",
-    "vows": ">= 0.5.8",
-    "winston": ">= 0.2.5"
+    "vows": ">= 0.5.8"
   },
   "main": "./lib/node-http-proxy",
   "bin": { "node-http-proxy": "./bin/node-http-proxy" },