Fix logs (#52)

Author: A3a3e1

CHANGELOG.md

@@ -1,6 +1,7 @@
 ## 1.1.7 (August 18, 2021)
 * Migrate from `request` to `axios` as main lib
 * Keep alive connection by default
+* Reduce logging level to trace
 
 ## 1.1.6 (July 29, 2021)
 * Now `getAttachment` from `AttachmentProcessor` may retrieve items from `Maester`

lib/authentication/CookieRestClient.ts

@@ -36,7 +36,7 @@ export class CookieRestClient extends NoAuthRestClient {
   }
 
   async login() {
-    this.emitter.logger.info('Performing Login ...');
+    this.emitter.logger.trace('Performing Login ...');
     const loginResponse = await this.requestCall({
       method: 'POST',
       url: this.cfg.loginUrl,
@@ -55,12 +55,12 @@ export class CookieRestClient extends NoAuthRestClient {
 
     this.handleLoginResponse(loginResponse);
     this.loggedIn = true;
-    this.emitter.logger.info('Login Complete.');
+    this.emitter.logger.trace('Login Complete.');
   }
 
   async logout() {
     if (this.cfg.logoutUrl && this.loggedIn) {
-      this.emitter.logger.info('Performing Logout...');
+      this.emitter.logger.trace('Performing Logout...');
       const logoutResponse = await this.requestCall({
         method: this.cfg.logoutMethod,
         url: this.cfg.logoutUrl,
@@ -71,9 +71,9 @@ export class CookieRestClient extends NoAuthRestClient {
       });
       this.handleLogoutResponse(logoutResponse);
       this.loggedIn = false;
-      this.emitter.logger.info('Logout complete.');
+      this.emitter.logger.trace('Logout complete.');
     } else {
-      this.emitter.logger.info('Nothing to logout');
+      this.emitter.logger.trace('Nothing to logout');
     }
   }
 

lib/authentication/NoAuthRestClient.ts

@@ -28,7 +28,7 @@ export class NoAuthRestClient {
       throw new Error(`Error in making request to ${response.config.url}/ Status code: ${response.statusCode}, Body: ${JSON.stringify(response.body)}`);
     }
 
-    this.emitter.logger.debug(`Response statusCode: ${response.statusCode}`);
+    this.emitter.logger.trace(`Response statusCode: ${response.statusCode}`);
     return response.body;
   }
 
@@ -41,18 +41,25 @@ export class NoAuthRestClient {
   //    if the provided URL is an absolute path. Defaults to true
   async makeRequest(options) {
     const {
-      url, method, body, headers = {}, urlIsSegment = true, isJson = true, responseHandler, axiosOptions = {},
+      axiosOptions = {},
+      body,
+      headers = {},
+      isJson = true,
+      method,
+      responseHandler,
+      url,
+      urlIsSegment = true,
     } = options;
     const urlToCall = urlIsSegment
       ? `${removeTrailingSlash(this.cfg.resourceServerUrl.trim())}/${removeLeadingSlash(url.trim())}` // Trim trailing or leading '/'
       : url.trim();
 
-    this.emitter.logger.debug(`Making ${method} request...`);
+    this.emitter.logger.trace(`Making ${method} request...`);
 
     const requestOptions = {
       ...axiosOptions,
-      method,
       headers,
+      method,
       data: body,
       url: urlToCall,
     };

lib/authentication/OAuth2AuthorizationCodeRestClient.ts

@@ -5,7 +5,7 @@ const request = promisify(require('request'));
 
 export class OAuth2RestClient extends NoAuthRestClient {
   private async fetchNewToken() {
-    this.emitter.logger.info('Fetching new token...');
+    this.emitter.logger.trace('Fetching new token...');
     const authTokenResponse = await request({
       uri: this.cfg.authorizationServerTokenEndpointUrl,
       method: 'POST',
@@ -21,7 +21,7 @@ export class OAuth2RestClient extends NoAuthRestClient {
       },
     });
 
-    this.emitter.logger.info('New token fetched...');
+    this.emitter.logger.trace('New token fetched...');
 
     if (authTokenResponse.statusCode >= 400) {
       throw new Error(`Error in authentication.  Status code: ${authTokenResponse.statusCode}, Body: ${JSON.stringify(authTokenResponse.body)}`);
@@ -37,7 +37,7 @@ export class OAuth2RestClient extends NoAuthRestClient {
     const tokenExpiryTime = new Date(this.cfg.oauth2.tokenExpiryTime);
     const now = new Date();
     if (now < tokenExpiryTime) {
-      this.emitter.logger.info('Previously valid token found.');
+      this.emitter.logger.trace('Previously valid token found.');
       return this.cfg.oauth2.access_token;
     }