Fix arg order for Storage Emulator command

firebase · Feb 18, 2022 · c1d4485 · c1d4485
1 parent aa178b2
commit c1d4485
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/emulator/downloadableEmulators.ts b/src/emulator/downloadableEmulators.ts
@@ -170,10 +170,10 @@ const Commands: { [s in DownloadableEmulators]: DownloadableEmulatorCommand } =
     // separately in ./storage/runtime.ts (not via the start function below).
     binary: "java",
     args: [
-      "-jar",
       // Required for rules error/warning messages, which are in English only.
       // Attempts to fetch the messages in another language leads to crashes.
       "-Duser.language=en",
+      "-jar",
       getExecPath(Emulators.STORAGE),
       "serve",
     ],

diff --git a/src/emulator/storage/rules/runtime.ts b/src/emulator/storage/rules/runtime.ts
@@ -155,6 +155,7 @@ export class StorageRulesRuntime {
     this._childprocess.stderr?.on("data", (buf: Buffer) => {
       const error = buf.toString();
       if (error.includes("jarfile")) {
+        EmulatorLogger.forEmulator(Emulators.STORAGE).log("ERROR", error);
         throw new FirebaseError(
           "There was an issue starting the rules emulator, please run 'firebase setup:emulators:storage` again"
         );