Fixed the response status when the parent was missing

OpenMarshal · Jun 28, 2017 · 056df31 · 056df31
1 parent 6fe546a
commit 056df31
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 2 deletions.
diff --git a/lib/server/v2/commands/Mkcol.js b/lib/server/v2/commands/Mkcol.js
@@ -13,7 +13,9 @@ var default_1 = (function () {
                     ctx.getResource(r.path.getParent(), function (e, rParent) {
                         rParent.type(function (e, parentType) {
                             if (e) {
-                                if (!ctx.setCodeFromError(e))
+                                if (e === Errors_1.Errors.ResourceNotFound)
+                                    ctx.setCode(WebDAVRequest_1.HTTPCodes.Conflict);
+                                else if (!ctx.setCodeFromError(e))
                                     ctx.setCode(WebDAVRequest_1.HTTPCodes.InternalServerError);
                                 return callback();
                             }

diff --git a/src/server/v2/commands/Mkcol.ts b/src/server/v2/commands/Mkcol.ts
@@ -15,7 +15,9 @@ export default class implements HTTPMethod
                         rParent.type((e, parentType) => {
                             if(e)
                             {
-                                if(!ctx.setCodeFromError(e))
+                                if(e === Errors.ResourceNotFound)
+                                    ctx.setCode(HTTPCodes.Conflict);
+                                else if(!ctx.setCodeFromError(e))
                                     ctx.setCode(HTTPCodes.InternalServerError)
                                 return callback();
                             }