clean up usage of logger, fix bug where logger is undefined

packages/rocketchat-api/server/api.js

@@ -5,7 +5,6 @@ const logger = new Logger('API', {});
 class API extends Restivus {
 	constructor(properties) {
 		super(properties);
-		this.logger = new Logger(`API ${ properties.version ? properties.version : 'default' } Logger`, {});
 		this.authMethods = [];
 		this.fieldSeparator = '.';
 		this.defaultFieldsToExclude = {
@@ -153,12 +152,12 @@ class API extends Restivus {
 							version,
 							entrypoint: route
 						});
-						this.logger.debug(`${ this.request.method.toUpperCase() }: ${ this.request.url }`);
+						logger.debug(`${ this.request.method.toUpperCase() }: ${ this.request.url }`);
 						let result;
 						try {
 							result = originalAction.apply(this);
 						} catch (e) {
-							this.logger.debug(`${ method } ${ route } threw an error:`, e.stack);
+							logger.debug(`${ method } ${ route } threw an error:`, e.stack);
 							result = RocketChat.API.v1.failure(e.message, e.error);
 						}
 
@@ -176,7 +175,7 @@ class API extends Restivus {
 					}
 
 					//Allow the endpoints to make usage of the logger which respects the user's settings
-					endpoints[method].logger = this.logger;
+					endpoints[method].logger = logger;
 				});
 			}