Improve emulator support for python functions.

firebase · Jan 24, 2023 · eb15111 · eb15111
1 parent 4f0adef
commit eb15111
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/emulator/functionsEmulator.ts b/src/emulator/functionsEmulator.ts
@@ -433,6 +433,7 @@ export class FunctionsEmulator implements EmulatorInstance {
           /.+?[\\\/]node_modules[\\\/].+?/, // Ignore node_modules
           /(^|[\/\\])\../, // Ignore files which begin the a period
           /.+\.log/, // Ignore files which have a .log extension
+          /.+?[\\\/]venv[\\\/].+?/, // Ignore site-packages in venv
         ],
         persistent: true,
       });
@@ -1320,6 +1321,10 @@ export class FunctionsEmulator implements EmulatorInstance {
     const childProcess = runWithVirtualEnv(args, backend.functionsDir, {
       ...process.env,
       ...envs,
+      // Required to flush stdout/stderr immediately to the piped channels.
+      PYTHONUNBUFFERED: "1",
+      // Required to prevent flask development server to reload on code changes.
+      DEBUG: "False",
       HOST: "127.0.0.1",
       PORT: port.toString(),
     });

diff --git a/src/functions/python.ts b/src/functions/python.ts
@@ -24,7 +24,7 @@ export function runWithVirtualEnv(
   return spawn(command, args, {
     shell: true,
     cwd,
-    stdio: [/* stdin= */ "ignore", /* stdout= */ "pipe", /* stderr= */ "pipe"],
+    stdio: [/* stdin= */ "pipe", /* stdout= */ "pipe", /* stderr= */ "pipe", "pipe"],
     // Linting disabled since internal types expect NODE_ENV which does not apply to Python runtimes.
     // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-explicit-any
     env: envs as any,