Repository navigation
ES2015 Deployed Code Crashes -- Optimizer not observing strict mode #19
Description
The serverless-optimizer-plugin does not appear to be observing 'use strict’; Thus, not defining a variable with var is ignored and when the code is deployed, it crashes in the Lambda environment.
To reproduce this issue:
1). Setup fresh project with a single component and function.
- Add the optimizer for es6 as outlined in readme.
"custom": {
"optimize": {
"exclude": [
"aws-sdk"
],
"includePaths": [
],
"transforms": [
{
"name": "babelify",
"opts": {
"presets": [
"es2015"
]
}
}
]
}
}
-
Make sure
'use strict;'is at the top of the function file. -
Add the following line to the function handler
foo = 1;
- Deploy and run function in AWS Lambda environment.
sls function run component/function -s dev
Check CloudWatch for Crash ReferenceError: foo is not defined.*
- Add
varahead of the variable:
var foo = 1;
- Deploy and run function again.
sls function run [path]
Works fine now.
Serverless deployment process should catch this error (due to strict mode) and fail. Not catching these errors results in unstable code being deployed that is out of spec.
If possible, adding babel-plugin-transform-strict-mode might help. How can this plugin be added to the optimizer json? It really shouldn't be required when strict is declared at the top of the file, but it would be nice to know how to include plugins for babel.
Activity
staceymoore commentedon Feb 16, 2016
@n8sabes try adding babel-plugin-transform-strict-mode to the transforms array in the same manner as babelify (see below for example and add opts if needed). That's what I did with my coffeeify transform and it worked fine. Be sure to install/save the package in the root node_modules of your sls project. Hope that helps.
n8sabes commentedon Feb 16, 2016
Thanks @staceymoore. I added the package to the project root and included the plugin to the transforms. Still no love, getting the following error with --debug option and not finding a good solution example (some gulp references). Did you configure or install anything else?
joostfarla commentedon Feb 19, 2016
Browserify doesn't seem to care about these kind of syntactic errors, at least it doesn't fail during the bundling process. However, it DOES include the "use strict" statement in the bundled package.
Some things you could do:
function runto find potential errors before deploying.n8sabes commentedon Feb 20, 2016
Hey @joostfarla -- Thanks for the conversation on the problem, and for your time looking at it.
Option 1:
function runis not really practical because it requires 100% code coverage on execution to catch syntactic errors such as missingvar. I have developed unit tests which are close to 100% code coverage, but sadly don't have every edge case. I also have the linter in WebStorm enabled so I can visually see syntactic errors, but this too is error prone for reliable deployments.Options 2: Your new JSHint plugin is a MUCH better idea to prevent bad deployments. Excited to try it and will let you know the outcome!
@staceymoore, thanks to you for your help on this topic as well.