Dependency Performance Improvements (middyjs#442)

* fix: move dep include out of middleware functions

* fix: only include relevent clients

* version bump

packages/http-urlencode-body-parser/index.js

@@ -4,23 +4,23 @@ const defaults = {
   extended: false
 }
 
-module.exports = (opts) => ({
-  before: (handler, next) => {
-    const options = Object.assign({}, defaults, opts)
+module.exports = (opts) => {
+  const options = Object.assign({}, defaults, opts)
+  const parserFn = options.extended ? require('qs').parse : require('querystring').decode
+  return {
+    before: (handler, next) => {
+      if (handler.event.headers) {
+        const contentTypeHeader = handler.event.headers['content-type'] || handler.event.headers['Content-Type']
+        if (contentTypeHeader) {
+          const { type } = contentType.parse(contentTypeHeader)
 
-    const parserFn = options.extended ? require('qs').parse : require('querystring').decode
-
-    if (handler.event.headers) {
-      const contentTypeHeader = handler.event.headers['content-type'] || handler.event.headers['Content-Type']
-      if (contentTypeHeader) {
-        const { type } = contentType.parse(contentTypeHeader)
-
-        if (type === 'application/x-www-form-urlencoded') {
-          handler.event.body = parserFn(handler.event.body)
+          if (type === 'application/x-www-form-urlencoded') {
+            handler.event.body = parserFn(handler.event.body)
+          }
         }
       }
-    }
 
-    next()
+      next()
+    }
   }
-})
+}

packages/ssm/__tests__/index.js

@@ -1,6 +1,6 @@
 jest.mock('aws-sdk')
 
-const { SSM } = require('aws-sdk')
+const SSM = require('aws-sdk/clients/ssm')
 const middy = require('../../core')
 const ssm = require('../')
 

packages/ssm/index.js

@@ -1,3 +1,4 @@
+const SSM = require('aws-sdk/clients/ssm')
 let ssmInstance
 
 module.exports = opts => {
@@ -31,7 +32,7 @@ module.exports = opts => {
         return next()
       }
 
-      ssmInstance = ssmInstance || getSSMInstance(options.awsSdkOptions)
+      ssmInstance = ssmInstance || new SSM(options.awsSdkOptions)
 
       const ssmPromises = Object.keys(options.paths).reduce(
         (aggregator, prefix) => {
@@ -144,23 +145,6 @@ const getTargetObjectToAssign = (handler, options) =>
 const getSSMParamValues = userParamsMap =>
   [...new Set(Object.keys(userParamsMap).map(key => userParamsMap[key]))]
 
-/**
- * Lazily load aws-sdk and initialize SSM constructor
- * to avoid performance penalties for those who doesn't use
- * this middleware. Sets ssmInstance var at the top of the module
- * or returns if it's already initialized
- * @param {Object} awsSdkOptions Options to use to initialize aws sdk constructor
- */
-const getSSMInstance = awsSdkOptions => {
-  // lazy load aws-sdk and SSM constructor to avoid performance
-  // penalties if you don't use this middleware
-
-  // AWS Lambda has aws-sdk included version 2.176.0
-  // see https://docs.aws.amazon.com/lambda/latest/dg/current-supported-versions.html
-  const { SSM } = require('aws-sdk')
-  return new SSM(awsSdkOptions)
-}
-
 /**
  * Throw error if SSM returns an error because we asked for params that don't exist
  * @throws {Error} When any invalid parameters found in response