fix(execute): parse stringified objects

Refs swagger-api/swagger-ui#7734
swagger-api · Apr 19, 2024 · 2702cf8 · 2702cf8
1 parent 7f142cd
commit 2702cf8
Show file tree

Hide file tree

Showing 2 changed files with 80 additions and 1 deletion.
diff --git a/src/http/index.js b/src/http/index.js
@@ -323,6 +323,14 @@ function formatKeyValueBySerializationOption(key, value, skipEncoding, serializa
   const encodeFn = (v) => valueEncoder(v, escape);
   const encodeKeyFn = skipEncoding ? (k) => k : (k) => encodeFn(k);
 
+  if (typeof value === 'string') {
+    try {
+      value = JSON.parse(value);
+    } catch {
+      // can't parse the value so treat it as as a simple string
+    }
+  }
+
   // Primitive
   if (typeof value !== 'object') {
     return [[encodeKeyFn(key), encodeFn(value)]];
@@ -442,7 +450,13 @@ export function mergeInQueryOrForm(req = {}) {
       req.formdata = formdata;
       req.body = formdata;
     } else {
-      req.body = encodeFormOrQuery(form);
+      let data = {};
+      if (typeof form === 'string') {
+        data = { form };
+      } else {
+        data = form;
+      }
+      req.body = encodeFormOrQuery(data);
     }
 
     delete req.form;

diff --git a/test/oas3/execute/main.js b/test/oas3/execute/main.js
@@ -871,6 +871,71 @@ describe('buildRequest - OpenAPI Specification 3.0', () => {
       });
     });
 
+    it('should serialize JSON values provided as objects in `deepObject` style', () => {
+      const req = buildRequest({
+        spec: {
+          openapi: '3.0.0',
+          paths: {
+            '/': {
+              post: {
+                operationId: 'myOp',
+                requestBody: {
+                  content: {
+                    'application/x-www-form-urlencoded': {
+                      schema: {
+                        type: 'object',
+                        properties: {
+                          a: {
+                            type: 'object',
+                            properties: {
+                              b: {
+                                type: 'string',
+                              },
+                              c: {
+                                type: 'array',
+                              },
+                              d: {
+                                type: 'object',
+                              },
+                            },
+                          },
+                        },
+                      },
+                      encoding: {
+                        a: {
+                          style: 'deepObject',
+                        },
+                      },
+                    },
+                  },
+                },
+              },
+            },
+          },
+        },
+        operationId: 'myOp',
+        requestBody: {
+          a: {
+            b: 'c',
+            c: ['d', 'e'],
+            d: {
+              e: 'f',
+            },
+          },
+        },
+      });
+
+      expect(req).toEqual({
+        method: 'POST',
+        url: `/`,
+        headers: {
+          'Content-Type': 'application/x-www-form-urlencoded',
+        },
+        credentials: 'same-origin',
+        body: 'a%5Bb%5D=c&a%5Bc%5D=%5B%22d%22%2C%22e%22%5D&a%5Bd%5D=%7B%22e%22%3A%22f%22%7D',
+      });
+    });
+
     it('should serialize JSON values provided as arrays of stringified objects', () => {
       const req = buildRequest({
         spec: {