Return 403 for non-exist container during blob SAS authentication

Azure · May 9, 2019 · a1d2695 · a1d2695
1 parent 4346aab
commit a1d2695
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 5 deletions.
diff --git a/.vscode/launch.json b/.vscode/launch.json
@@ -10,7 +10,7 @@
       "name": "Azurite Blob Service",
       "cwd": "${workspaceFolder}",
       "runtimeArgs": ["-r", "ts-node/register"],
-      "args": ["${workspaceFolder}/src/blob/main.ts"],
+      "args": ["${workspaceFolder}/src/blob/main.ts", "-d", "debug.log"],
       "outputCapture": "std"
     },
     {

diff --git a/src/blob/authentication/BlobSASAuthenticator.ts b/src/blob/authentication/BlobSASAuthenticator.ts
@@ -50,9 +50,11 @@ export default class BlobSASAuthenticator implements IAuthenticator {
 
     const containerName = blobContext.container;
     if (containerName === undefined) {
-      throw RangeError(
-        `BlobSASAuthenticator:validate() container name is undefined in context.`
+      this.logger.error(
+        `BlobSASAuthenticator:validate() container name is undefined in context.`,
+        context.contextID
       );
+      return undefined;
     }
 
     const blobName = blobContext.blob;

diff --git a/src/blob/main.ts b/src/blob/main.ts
@@ -61,15 +61,15 @@ async function main() {
   const beforeCloseMessage = `Azurite Blob service is closing...`;
   const afterCloseMessage = `Azurite Blob service successfully closed`;
   process
-    .on("message", msg => {
+    .once("message", msg => {
       if (msg === "shutdown") {
         console.log(beforeCloseMessage);
         server.close().then(() => {
           console.log(afterCloseMessage);
         });
       }
     })
-    .on("SIGINT", () => {
+    .once("SIGINT", () => {
       console.log(beforeCloseMessage);
       server.close().then(() => {
         console.log(afterCloseMessage);