fix regression where env is ignored

include/zapierwrapper.js

@@ -2,10 +2,5 @@
 const path = require('path');
 const zapier = require('zapier-platform-core');
 const appPath = path.resolve(__dirname, 'index.js');
-let opts;
-try {
-  opts = require(appPath).flags;
-} catch (error) {
-  // nothing to see here
-}
-module.exports = { handler: zapier.createAppHandler(appPath, opts) };
+// don't `require(appPath)` out here
+module.exports = { handler: zapier.createAppHandler(appPath) };

src/tools/create-app-tester.js

@@ -37,8 +37,7 @@ const promisifyHandler = handler => {
 
 // A shorthand compatible wrapper for testing.
 const createAppTester = (appRaw, { customStoreKey } = {}) => {
-  const opts = appRaw.flags;
-  const handler = createLambdaHandler(appRaw, opts);
+  const handler = createLambdaHandler(appRaw);
   const createHandlerPromise = promisifyHandler(handler);
 
   const randomSeed = genId();

src/tools/create-lambda-handler.js

@@ -130,14 +130,8 @@ const loadApp = (event, rpc, appRawOrPath) => {
   });
 };
 
-const createLambdaHandler = (appRawOrPath, { skipHttpPatch } = {}) => {
+const createLambdaHandler = appRawOrPath => {
   const handler = (event, context, callback) => {
-    // Adds logging for _all_ kinds of http(s) requests, no matter the library
-    if (!skipHttpPatch) {
-      const httpPatch = createHttpPatch(event);
-      httpPatch(require('http')); // 'https' uses 'http' under the hood
-    }
-
     // Wait for all async events to complete before callback returns.
     // This is not strictly necessary since this is the default now when
     // using the callback; just putting it here to be explicit.
@@ -200,6 +194,13 @@ const createLambdaHandler = (appRawOrPath, { skipHttpPatch } = {}) => {
         .then(appRaw => {
           const app = createApp(appRaw);
 
+          const { skipHttpPatch } = appRaw.flags || {};
+          // Adds logging for _all_ kinds of http(s) requests, no matter the library
+          if (!skipHttpPatch) {
+            const httpPatch = createHttpPatch(event);
+            httpPatch(require('http')); // 'https' uses 'http' under the hood
+          }
+
           // TODO: Avoid calling prepareApp(appRaw) repeatedly here as createApp()
           // already calls prepareApp() but just doesn't return it.
           const compiledApp = schemaTools.prepareApp(appRaw);