Closed nelsonic closed 8 years ago
The code in the re-written class (with conditional assignment of context or config will *always throw error because invokedFunctionArn will be invalid by definition. see: https://github.com/tcdl/aws-lambda-helper/blob/d692d49dd2003ad4eb6a8fe911705e6c67888ca0/lib/index.js#L18-L20
context
config
throw error
invokedFunctionArn
So not setting the context/config will always result in throwing an error. So... should we even have the conditional assignment of an empty context?
This is no longer an issue in the latest version. https://github.com/numo-labs/aws-lambda-helper/blob/7c1f5d64155795b7975d340ae8dd81e2b1e134c9/lib/index.js#L20
The code in the re-written class (with conditional assignment of
context
orconfig
will *alwaysthrow error
becauseinvokedFunctionArn
will be invalid by definition. see: https://github.com/tcdl/aws-lambda-helper/blob/d692d49dd2003ad4eb6a8fe911705e6c67888ca0/lib/index.js#L18-L20So not setting the context/config will always result in throwing an error. So... should we even have the conditional assignment of an empty
context
?