Merge 002b3ec into da17f14

firebase · May 13, 2019 · e9e0e8a · e9e0e8a
2 parents da17f14 + 002b3ec
commit e9e0e8a
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 2 deletions.
diff --git a/changelog.txt b/changelog.txt
@@ -0,0 +1 @@
+fixed - Fixed issue with CORS rejecting some callable functions.
diff --git a/src/emulator/functionsEmulator.ts b/src/emulator/functionsEmulator.ts
@@ -90,9 +90,17 @@ export class FunctionsEmulator implements EmulatorInstance {
 
     hub.use((req, res, next) => {
       // Allow CORS to facilitate easier testing.
-      // Source: https://enable-cors.org/server_expressjCannot understand what targets to deploys.html
+      // Sources:
+      //  * https://enable-cors.org/server_expressjCannot understand what targets to deploys.html
+      //  * https://stackoverflow.com/a/37228330/324977
       res.header("Access-Control-Allow-Origin", "*");
-      res.header("Access-Control-Allow-Headers", "Origin, X-Requested-With, Content-Type, Accept");
+
+      // Callable functions send "Authorization" and "Content-Type".
+      res.header(
+        "Access-Control-Allow-Headers",
+        "Origin, X-Requested-With, Content-Type, Authorization, Accept"
+      );
+      res.header("Access-Control-Allow-Methods", "GET,OPTIONS,POST");
 
       let data = "";
       req.on("data", (chunk: any) => {