revert(amplify-provider-awscloudformation): removed cfn logging (#6119)

Author: ammarkarachi

packages/amplify-provider-awscloudformation/src/aws-utils/aws-cfn.js

@@ -11,8 +11,6 @@ const providerName = require('../constants').ProviderName;
 const { formUserAgentParam } = require('./user-agent');
 const configurationManager = require('../configuration-manager');
 const { stateManager } = require('amplify-cli-core');
-const { fileLogger } = require('../utils/aws-logger');
-const logger = fileLogger('aws-cfn');
 
 const CFN_MAX_CONCURRENT_REQUEST = 5;
 const CFN_POLL_TIME = 5 * 1000; // 5 secs wait to check if  new stacks are created by root stack
@@ -59,10 +57,8 @@ class CloudFormation {
     self.eventStartTime = new Date();
 
     return new Promise((resolve, reject) => {
-      logger('cfnModel.createStack', [cfnParentStackParams])();
       cfnModel.createStack(cfnParentStackParams, createErr => {
         this.readStackEvents(cfnParentStackParams.StackName);
-        logger('cfnModel.createStack', [cfnParentStackParams])(createErr);
         if (createErr) {
           context.print.error('An error occurred when creating the CloudFormation stack');
           reject(createErr);
@@ -177,9 +173,6 @@ class CloudFormation {
 
   getStackEvents(stackName) {
     const self = this;
-    const describeStackEventsArgs = { StackName: stackName };
-    const log = logger('getStackEvents.cfnModel.describeStackEvents', [describeStackEventsArgs]);
-    log();
     return this.cfn
       .describeStackEvents({ StackName: stackName })
       .promise()
@@ -189,7 +182,6 @@ class CloudFormation {
         return Promise.resolve(events);
       })
       .catch(e => {
-        log(e);
         if (e && e.code === 'Throttling') {
           return Promise.resolve([]);
         }
@@ -231,7 +223,6 @@ class CloudFormation {
           Body: fs.createReadStream(filePath),
           Key: cfnFile,
         };
-        logger('updateResourceStack.s3.uploadFile', [{ Key: s3Params.cfnFile }])();
         return s3.uploadFile(s3Params, false);
       })
       .then(bucketName => {
@@ -244,7 +235,6 @@ class CloudFormation {
         const self = this;
         this.eventStartTime = new Date();
         return new Promise((resolve, reject) => {
-          logger('updateResourceStack.describeStack', [cfnStackCheckParams])();
           this.describeStack(cfnStackCheckParams)
             .then(() => {
               const cfnParentStackParams = {
@@ -267,7 +257,6 @@ class CloudFormation {
                 ],
                 Tags,
               };
-              logger('updateResourceStack.updateStack', [cfnStackCheckParams])();
               cfnModel.updateStack(cfnParentStackParams, updateErr => {
                 self.readStackEvents(stackName);
 
@@ -303,7 +292,6 @@ class CloudFormation {
     };
     const projectDetails = this.context.amplify.getProjectDetails();
     const { amplifyMeta } = projectDetails;
-    logger('updateamplifyMetaFileWithStackOutputs.cfn.describeStackResources', [cfnParentStackParams])();
     const result = await this.cfn.describeStackResources(cfnParentStackParams).promise();
     const resources = result.StackResources.filter(
       resource =>
@@ -368,12 +356,9 @@ class CloudFormation {
   }
 
   listExports(nextToken = null) {
-    const log = logger('listExports.cfn.listExports', [{ NextToken: nextToken }]);
     return new Promise((resolve, reject) => {
-      log();
       this.cfn.listExports(nextToken ? { NextToken: nextToken } : {}, (err, data) => {
         if (err) {
-          log(err);
           reject(err);
         } else if (data.NextToken) {
           this.listExports(data.NextToken).then(innerExports => resolve([...data.Exports, ...innerExports]));
@@ -386,15 +371,12 @@ class CloudFormation {
 
   describeStack(cfnNestedStackParams, maxTry = 10, timeout = CFN_POLL_TIME) {
     const cfnModel = this.cfn;
-    const log = logger('describeStack.cfn.describeStacks', [cfnNestedStackParams]);
     return new Promise((resolve, reject) => {
-      log();
       cfnModel
         .describeStacks(cfnNestedStackParams)
         .promise()
         .then(result => resolve(result))
         .catch(e => {
-          log(e);
           if (e.code === 'Throttling' && e.retryable) {
             setTimeout(() => {
               resolve(this.describeStack(cfnNestedStackParams, maxTry - 1, timeout));
@@ -419,10 +401,8 @@ class CloudFormation {
     };
 
     const cfnModel = this.cfn;
-    const log = logger('deleteResourceStack.cfn.describeStacks', [cfnStackParams]);
 
     return new Promise((resolve, reject) => {
-      log();
       cfnModel.describeStacks(cfnStackParams, (err, data) => {
         const cfnDeleteStatus = 'stackDeleteComplete';
         if (
@@ -448,7 +428,6 @@ class CloudFormation {
             });
           });
         } else {
-          log(err);
           reject(err);
         }
       });