Merge pull request motdotla#794 from abetomo/migration-to-aws-sdk-v3-drop-v2

sdk-v3: Drop SDK V2

Author: DeviaVir

lib/aws.js

@@ -1,38 +1,41 @@
-'use strict'
-
-const aws = require('aws-sdk')
-const proxy = require('proxy-agent')
-
-module.exports = {
-  sdk: aws,
-  updateConfig (config, region) {
-    const awsSecurity = { region }
-
-    if (config.profile) {
-      aws.config.credentials = new aws.SharedIniFileCredentials({
-        profile: config.profile
-      })
-    } else {
-      awsSecurity.accessKeyId = config.accessKey
-      awsSecurity.secretAccessKey = config.secretKey
-    }
-
-    if (config.sessionToken) {
-      awsSecurity.sessionToken = config.sessionToken
-    }
-
-    if (config.deployTimeout) {
-      aws.config.httpOptions.timeout = parseInt(config.deployTimeout)
-    }
-
-    if (config.proxy) {
-      aws.config.httpOptions.agent = proxy(config.proxy)
-    }
-
-    if (config.endpoint) {
-      aws.config.endpoint = config.endpoint
-    }
-
-    aws.config.update(awsSecurity)
-  }
-}
+// TODO:
+// Support these option settings in SDK v3
+//
+// 'use strict'
+//
+// const aws = require('aws-sdk')
+// const proxy = require('proxy-agent')
+//
+// module.exports = {
+//   sdk: aws,
+//   updateConfig (config, region) {
+//     const awsSecurity = { region }
+//
+//     if (config.profile) {
+//       aws.config.credentials = new aws.SharedIniFileCredentials({
+//         profile: config.profile
+//       })
+//     } else {
+//       awsSecurity.accessKeyId = config.accessKey
+//       awsSecurity.secretAccessKey = config.secretKey
+//     }
+//
+//     if (config.sessionToken) {
+//       awsSecurity.sessionToken = config.sessionToken
+//     }
+//
+//     if (config.deployTimeout) {
+//       aws.config.httpOptions.timeout = parseInt(config.deployTimeout)
+//     }
+//
+//     if (config.proxy) {
+//       aws.config.httpOptions.agent = proxy(config.proxy)
+//     }
+//
+//     if (config.endpoint) {
+//       aws.config.endpoint = config.endpoint
+//     }
+//
+//     aws.config.update(awsSecurity)
+//   }
+// }

lib/main.js

@@ -3,7 +3,6 @@
 const process = require('process')
 const path = require('path')
 const os = require('os')
-const aws = require(path.join(__dirname, 'aws'))
 const { exec, execSync, execFile } = require('child_process')
 const fs = require('fs-extra')
 const klaw = require('klaw')
@@ -957,7 +956,7 @@ they may not work as expected in the Lambda environment.
 
   async _deployToRegion (program, params, region, buffer) {
     // sdk v3 todo: Migration of aws.updateConfig.
-    aws.updateConfig(program, region)
+    // aws.updateConfig(program, region)
     const config = { region }
 
     console.log('=> Reading event source file to memory')