fix(body-data): detect YAML with /x-yaml content type (#1171)

fixes #1170

Author: dominique-pfister

packages/helix-shared-body-data/src/body-data-wrapper.js

@@ -32,7 +32,7 @@ async function getData(request, opts) {
   }
 
   const { supportYAML } = opts;
-  if (supportYAML && /\/yaml/.test(contentType) && BODY_METHODS.includes(request.method)) {
+  if (supportYAML && /\/(x-)?yaml/.test(contentType) && BODY_METHODS.includes(request.method)) {
     return request.text();
   }
 

packages/helix-shared-body-data/test/body-data-wrapper.test.js

@@ -273,24 +273,26 @@ describe('Body Data Wrapper Unit Tests (YAML Body)', () => {
   const contents = { indices: [] };
 
   ['POST', 'post', 'PUT', 'PATCH'].forEach((method) => {
-    it(`Loads YAML (${method})`, async () => {
-      const universalfunct = async (request, context) => {
-        const yaml = YAML.parse(context.data);
-        assert.deepStrictEqual(yaml, contents);
-        return new Response('ok');
-      };
-
-      const actualfunct = wrap(universalfunct).with(bodyData, { supportYAML: true });
-      const response = await actualfunct(new Request('http://localhost', {
-        body: YAML.stringify(contents),
-        method,
-        headers: {
-          'content-type': 'text/yaml',
-        },
-      }), {
-        log,
+    ['text/yaml', 'application/x-yaml'].forEach((yamlType) => {
+      it(`Loads YAML with type ${yamlType} (${method})`, async () => {
+        const universalfunct = async (request, context) => {
+          const yaml = YAML.parse(context.data);
+          assert.deepStrictEqual(yaml, contents);
+          return new Response('ok');
+        };
+
+        const actualfunct = wrap(universalfunct).with(bodyData, { supportYAML: true });
+        const response = await actualfunct(new Request('http://localhost', {
+          body: YAML.stringify(contents),
+          method,
+          headers: {
+            'content-type': yamlType,
+          },
+        }), {
+          log,
+        });
+        assert.strictEqual(response.status, 200, 'universal function should be executed');
       });
-      assert.strictEqual(response.status, 200, 'universal function should be executed');
     });
   });