removing errant console.log, update lint rules

Author: shellscape

lib/Server.js

@@ -1,6 +1,6 @@
 'use strict';
 
-/* eslint no-console: off, func-names: off */
+/* eslint func-names: off */
 require('./polyfills');
 
 const fs = require('fs');
@@ -24,6 +24,7 @@ const OptionsValidationError = require('./OptionsValidationError');
 const optionsSchema = require('./optionsSchema.json');
 
 const clientStats = { errorDetails: false };
+const log = console.log; // eslint-disable-line no-console
 
 function Server(compiler, options) {
   // Default options
@@ -275,8 +276,8 @@ function Server(compiler, options) {
           app.get('*', express.static(item));
         });
       } else if (/^(https?:)?\/\//.test(contentBase)) {
-        console.log('Using a URL as contentBase is deprecated and will be removed in the next major version. Please use the proxy option instead.');
-        console.log('proxy: {\n\t"*": "<your current contentBase configuration>"\n}'); // eslint-disable-line quotes
+        log('Using a URL as contentBase is deprecated and will be removed in the next major version. Please use the proxy option instead.');
+        log('proxy: {\n\t"*": "<your current contentBase configuration>"\n}'); // eslint-disable-line quotes
         // Redirect every request to contentBase
         app.get('*', (req, res) => {
           res.writeHead(302, {
@@ -285,8 +286,8 @@ function Server(compiler, options) {
           res.end();
         });
       } else if (typeof contentBase === 'number') {
-        console.log('Using a number as contentBase is deprecated and will be removed in the next major version. Please use the proxy option instead.');
-        console.log('proxy: {\n\t"*": "//localhost:<your current contentBase configuration>"\n}'); // eslint-disable-line quotes
+        log('Using a number as contentBase is deprecated and will be removed in the next major version. Please use the proxy option instead.');
+        log('proxy: {\n\t"*": "//localhost:<your current contentBase configuration>"\n}'); // eslint-disable-line quotes
         // Redirect every request to the port contentBase
         app.get('*', (req, res) => {
           res.writeHead(302, {
@@ -384,14 +385,14 @@ function Server(compiler, options) {
 
         // cert is more than 30 days old, kill it with fire
         if ((now - certStat.ctime) / certTtl > 30) {
-          console.log('SSL Certificate is more than 30 days old. Removing.');
+          log('SSL Certificate is more than 30 days old. Removing.');
           del.sync([certPath], { force: true });
           certExists = false;
         }
       }
 
       if (!certExists) {
-        console.log('Generating SSL Certificate');
+        log('Generating SSL Certificate');
         const attrs = [{ name: 'commonName', value: 'localhost' }];
         const pems = selfsigned.generate(attrs, {
           algorithm: 'sha256',
@@ -549,13 +550,12 @@ Server.prototype.listen = function (port, hostname, fn) {
       // Limit useless logs
       log(severity, line) {
         if (severity === 'error') {
-          console.log(line);
+          log(line);
         }
       }
     });
 
     sockServer.on('connection', (conn) => {
-      console.log('connection.remotePort', conn.remotePort);
       if (!conn) return;
       if (!this.checkHost(conn.headers)) {
         this.sockWrite([conn], 'error', 'Invalid Host header');