Merge pull request #39 from zebbra/nested-params

Support nested objects in query params
moleculerjs · Apr 24, 2018 · 3299c4c · 3299c4c
2 parents 10bdec9 + 98fdf5d
commit 3299c4c
Show file tree

Hide file tree

Showing 4 changed files with 37 additions and 3 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -61,6 +61,7 @@
     "lodash": "4.17.5",
     "nanomatch": "1.2.9",
     "path-to-regexp": "1.7.0",
+    "qs": "6.5.1",
     "serve-static": "1.13.2"
   },
   "engines": {

diff --git a/src/index.js b/src/index.js
@@ -8,7 +8,7 @@
 
 const http 						= require("http");
 const https 					= require("https");
-const queryString 				= require("querystring");
+const queryString 				= require("qs");
 const deprecate 				= require("depd")("moleculer-web");
 
 const _ 						= require("lodash");

diff --git a/test/integration/index.spec.js b/test/integration/index.spec.js
@@ -789,6 +789,32 @@ describe("Test aliases", () => {
 			});
 	});
 
+	it("GET opt-test with array params", () => {
+		return request(server)
+			.get("/api/opt-test")
+			.query("a=1&a=2")
+			.expect(200)
+			.expect("Content-Type", "application/json; charset=utf-8")
+			.then(res => {
+				expect(res.body.params).toEqual({
+					a: ["1", "2"],
+				});
+			});
+	});
+
+	it("GET opt-test with nested params", () => {
+		return request(server)
+			.get("/api/opt-test")
+			.query("foo[bar]=a&foo[bar]=b&foo[baz]=c")
+			.expect(200)
+			.expect("Content-Type", "application/json; charset=utf-8")
+			.then(res => {
+				expect(res.body.params).toEqual({
+					foo: { bar: ["a", "b"], baz: "c" }
+				});
+			});
+	});
+
 	it("GET opt-test/:name? without name", () => {
 		return request(server)
 			.get("/api/opt-test")