Merge pull request #16 from curtislacy/Remove-Debug

Curtis · Curtis · commit add27419d08a · 2014-06-16T11:09:52.000-04:00
Removed some console.logs which could dump passwords into log files.
diff --git a/lib/ImapStrategy.js b/lib/ImapStrategy.js
@@ -194,7 +194,6 @@ ImapStrategy.prototype.testAuthentication = function( owner, account, callback )
 			finished = true;
 			if( error )
 			{
-				console.log( error );
 				if( error.code == 'AUTHENTICATIONFAILED' )
 					callback( null, false );
 				else
diff --git a/server.js b/server.js
@@ -150,8 +150,7 @@ app.configure(function () {
 		{
 			req.getAuthTokens = function() { return false; };
 		}
-		console.log('>>>', req.isAuthenticated(), req.user);
-	next();
+		next();
 	} );
 	app.use(express.static(__dirname+'/dist'));
 

Original file line number	Diff line number	Diff line change
`@@ -194,7 +194,6 @@ ImapStrategy.prototype.testAuthentication = function( owner, account, callback )`
`194`	`194`	`finished = true;`
`195`	`195`	`if( error )`
`196`	`196`	`{`
`197`		`- console.log( error );`
`198`	`197`	`if( error.code == 'AUTHENTICATIONFAILED' )`
`199`	`198`	`callback( null, false );`
`200`	`199`	`else`
Original file line number	Diff line number	Diff line change
`@@ -150,8 +150,7 @@ app.configure(function () {`
`150`	`150`	`{`
`151`	`151`	`req.getAuthTokens = function() { return false; };`
`152`	`152`	`}`
`153`		`- console.log('>>>', req.isAuthenticated(), req.user);`
`154`		`- next();`
	`153`	`+ next();`
`155`	`154`	`} );`
`156`	`155`	`app.use(express.static(__dirname+'/dist'));`
`157`	`156`