Show distinct error message when content type is unspecified vs. specified but unknown; fixes #186.

Author: coddingtonbear

src/constants.ts

@@ -13,6 +13,8 @@ export const ERROR_CODE_MESSAGES: Record<ErrorCode, string> = {
     "Authorization required.  Find your API Key in the 'Local REST API' section of your Obsidian settings.",
   [ErrorCode.ContentTypeSpecificationRequired]:
     "Content-Type header required; this API accepts data in multiple content-types and you must indicate the content-type of your request body via the Content-Type header.",
+  [ErrorCode.InvalidContentType]:
+    "Unknown or invalid Content-Type specified in Content-Type header.",
   [ErrorCode.InvalidContentInsertionPositionValue]:
     "Invalid 'Content-Insertion-Position' header value.",
   [ErrorCode.InvalidContentForContentType]:

src/requestHandler.ts

@@ -1114,11 +1114,16 @@ export default class RequestHandler {
     };
     const contentType = req.headers["content-type"];
 
-    if (!handlers[contentType]) {
+    if (!contentType) {
       this.returnCannedResponse(res, {
         errorCode: ErrorCode.ContentTypeSpecificationRequired,
       });
       return;
+    } else if (!handlers[contentType]) {
+      this.returnCannedResponse(res, {
+        errorCode: ErrorCode.InvalidContentType,
+      });
+      return;
     }
 
     try {

src/types.ts

@@ -5,6 +5,7 @@ import { IPeriodicNoteSettings } from "obsidian-daily-notes-interface";
 export enum ErrorCode {
   TextContentEncodingRequired = 40010,
   ContentTypeSpecificationRequired = 40011,
+  InvalidContentType = 40012,
   InvalidContentForContentType = 40015,
   InvalidContentInsertionPositionValue = 40050,
   MissingHeadingHeader = 40051,