chore: removed comments

Author: Vieltojarvi

packages/amplify-category-auth/provider-utils/awscloudformation/triggers/CustomMessage/assets/verify.js

@@ -37,8 +37,6 @@ function confirm() {
     Username: userName,
   };
 
-  // AWS.config.region = region;
-
   var cognitoidentityserviceprovider = new AWS.CognitoIdentityServiceProvider({ region: region });
 
   cognitoidentityserviceprovider.confirmSignUp(params, function (err, data) {

packages/amplify-category-hosting/__mocks__/mockAwsProviderModule.js

@@ -1,12 +1,3 @@
-// async function getConfiguredAWSClient() {
-//   return {
-//     S3,
-//     IAM,
-//     Pinpoint,
-//     CloudFront,
-//   };
-// }
-
 async function getConfiguredAWSClient() {
   return {
     credentials: 'some credentials',
@@ -18,31 +9,6 @@ async function getConfiguredPinpointClient() {
   return new Pinpoint();
 }
 
-class S3 {
-  upload() {
-    return {
-      promise: () => Promise.resolve({}),
-    };
-  }
-}
-
-class IAM {
-  createPolicy() {
-    return {
-      promise: () =>
-        Promise.resolve({
-          Policy: {},
-        }),
-    };
-  }
-
-  attachRolePolicy() {
-    return {
-      promise: () => Promise.resolve({}),
-    };
-  }
-}
-
 class Pinpoint {
   constructor() {
     this.config = {};
@@ -78,14 +44,6 @@ class Pinpoint {
   }
 }
 
-class CloudFront {
-  createInvalidation() {
-    return {
-      promise: () => Promise.resolve({}),
-    };
-  }
-}
-
 module.exports = {
   getConfiguredAWSClient,
   getConfiguredPinpointClient,

packages/amplify-category-notifications/src/pinpoint-helper.ts

@@ -548,11 +548,6 @@ export const getPinpointClient = async (
   // HTTP_PROXY & HTTPS_PROXY env vars are read automatically by ProxyAgent, but we check to see if they are set before using the proxy
   if (httpProxy) {
     httpAgent = new ProxyAgent();
-    // aws.config.update({
-    //   httpOptions: {
-    //     agent: new ProxyAgent(),
-    //   },
-    // });
   }
 
   return new aws.Pinpoint({ ...cred, ...defaultOptions, httpOptions: { agent: httpAgent } });

packages/amplify-e2e-tests/src/__tests__/storage-simulator/S3server.test.ts

@@ -17,12 +17,6 @@ let simulator;
 jest.setTimeout(2000000);
 
 beforeAll(async () => {
-  // AWS.config.update({
-  //   accessKeyId: 'fakeaccesskeyidfortesting',
-  //   secretAccessKey: 'fakeaccesssecretkeyfortesting',
-  //   region: 'eu-west-2',
-  // });
-
   const ep = new AWS.Endpoint('http://localhost:20005');
   s3client = new AWS.S3({
     accessKeyId: fakeAccessId,

packages/amplify-e2e-tests/src/import-helpers/utilities.ts

@@ -296,7 +296,6 @@ const addAppClient = async (
   const projectDetails = getProjectMeta(projectRoot);
   const authDetails = getAuthProjectDetails(projectRoot);
   const creds = new aws.SharedIniFileCredentials({ profile: profileName });
-  //aws.config.credentials = creds;
 
   const cognitoClient = new aws.CognitoIdentityServiceProvider({
     credentials: creds,
@@ -335,7 +334,6 @@ export const deleteAppClient = async (profileName: string, projectRoot: string,
   const authDetails = getAuthProjectDetails(projectRoot);
   const projectDetails = getProjectMeta(projectRoot);
   const creds = new aws.SharedIniFileCredentials({ profile: profileName });
-  //aws.config.credentials = creds;
 
   const cognitoClient = new aws.CognitoIdentityServiceProvider({
     credentials: creds,

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

@@ -1,13 +1,11 @@
 const aws = require('aws-sdk');
-// const { ProxyAgent } = require('proxy-agent');
 const configurationManager = require('../configuration-manager');
 const { regions: amplifyServiceRegions } = require('../aws-regions');
 const { proxyAgent } = require('./aws-globals');
 
 async function getConfiguredAmplifyClient(context, options = {}) {
   let cred = {};
   let defaultOptions = {};
-  // const httpProxy = process.env.HTTP_PROXY || process.env.HTTPS_PROXY;
   const envVarEndpoint = process.env.AWS_AMPLIFY_ENDPOINT;
 
   try {
@@ -22,15 +20,6 @@ async function getConfiguredAmplifyClient(context, options = {}) {
     };
   }
 
-  // **affected by SDK migrations
-  // if (httpProxy) {
-  //   aws.config.update({
-  //     httpOptions: {
-  //       agent: new ProxyAgent(),
-  //     },
-  //   });
-  // }
-
   const config = {
     ...cred,
     ...defaultOptions,

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

@@ -8,8 +8,6 @@ const configFilePath = path.join(dotAWSDirPath, 'config');
 
 let aws;
 
-// **affected by SDK migrations
-// this may need to be entirely gutted
 try {
   delete require.cache[require.resolve('aws-sdk')];
   if (fs.existsSync(credentialsFilePath) && fs.existsSync(configFilePath)) {
@@ -24,24 +22,7 @@ try {
   aws = require('aws-sdk');
 }
 
-const { ProxyAgent } = require('proxy-agent');
-const configurationManager = require('../configuration-manager');
-
 aws.configureWithCreds = async (context) => {
-  // const httpProxy = process.env.HTTP_PROXY || process.env.HTTPS_PROXY;
-  // const config = await configurationManager.loadConfiguration(context, aws);
-  // if (config) {
-  //   aws.config.update(config);
-  // }
-
-  // if (httpProxy) {
-  //   aws.config.update({
-  //     httpOptions: {
-  //       agent: new ProxyAgent(httpProxy),
-  //     },
-  //   });
-  // }
-
   return aws;
 };
 

packages/amplify-provider-awscloudformation/src/utils/admin-helpers.ts

@@ -73,8 +73,6 @@ export async function getTempCredsWithAdminTokens(context: $TSContext, appId: st
   // use tokens to get creds and assign to config
   const awsConfigInfo = await getAdminCognitoCredentials(idToken, IdentityId, region);
 
-  // aws.config.update(awsConfigInfo);
-
   // need to use Cognito creds to get STS creds - otherwise
   // users will not be able to provision Cognito resources
   const sts = new aws.STS({