issue apache#1728 Using const or let in global scope have problem in …

…nodejs actions (apache#1929)
houshengbo · Mar 10, 2017 · 2ea3146 · 2ea3146
1 parent f2c654e
commit 2ea3146
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/core/nodejsActionBase/runner.js b/core/nodejsActionBase/runner.js
@@ -72,8 +72,7 @@ function NodeActionRunner() {
         } else {
             // The code is a plain old JS file.
             try {
-                eval(message.code);
-                thisRunner.userScriptMain = eval(message.main);
+                thisRunner.userScriptMain = eval('(function(){' + message.code + '\nreturn ' + message.main + '})()');
                 assertMainIsFunction();
                 // See comment above about 'true'; it has no specific meaning.
                 return Promise.resolve(true);