fix: Invalid query format: \"filters[0]\" does not match any of the allowed types if queryRewrite is using non string values

Author: paveltiunov

packages/cubejs-api-gateway/src/query.js

@@ -70,7 +70,7 @@ const oneFilter = Joi.object().keys({
   dimension: id,
   member: id,
   operator: Joi.valid(...operators).required(),
-  values: Joi.array().items(Joi.string().allow('', null), Joi.link('...'))
+  values: Joi.array().items(Joi.string().allow('', null), Joi.link('...'), Joi.number(), Joi.boolean())
 }).xor('dimension', 'member');
 
 const oneCondition = Joi.object().keys({
@@ -200,18 +200,19 @@ const normalizeQuery = (query, persistent) => {
     order: normalizeQueryOrder(query.order),
     filters: (query.filters || []).map(f => {
       const { dimension, member, ...filter } = f;
-      const normalizedFlter = {
+      const normalizedFilter = {
         ...filter,
-        member: member || dimension
+        member: member || dimension,
+        values: filter.values?.map(v => (v != null ? v.toString() : v))
       };
 
-      Object.defineProperty(normalizedFlter, 'dimension', {
+      Object.defineProperty(normalizedFilter, 'dimension', {
         get() {
           console.warn('Warning: Attribute `filter.dimension` is deprecated. Please use \'member\' instead of \'dimension\'.');
           return this.member;
         }
       });
-      return normalizedFlter;
+      return normalizedFilter;
     }),
     dimensions: (query.dimensions || []).filter(d => d.split('.').length !== 3),
     timeDimensions: (query.timeDimensions || []).map(td => {

packages/cubejs-api-gateway/test/index.test.ts

@@ -284,6 +284,51 @@ describe('API Gateway', () => {
     );
   });
 
+  test('normalize filter number values', async () => {
+    const { app } = createApiGateway();
+
+    const query = {
+      measures: ['Foo.bar'],
+      filters: [{
+        member: 'Foo.bar',
+        operator: 'gte',
+        values: [10.5]
+      }, {
+        member: 'Foo.bar',
+        operator: 'gte',
+        values: [0]
+      }]
+    };
+
+    return requestBothGetAndPost(
+      app,
+      { url: '/cubejs-api/v1/dry-run', query: { query: JSON.stringify(query) }, body: { query } },
+      (res) => {
+        expect(res.body.normalizedQueries).toStrictEqual([
+          {
+            measures: ['Foo.bar'],
+            timezone: 'UTC',
+            order: [],
+            filters: [{
+              member: 'Foo.bar',
+              operator: 'gte',
+              values: ['10.5']
+            }, {
+              member: 'Foo.bar',
+              operator: 'gte',
+              values: ['0']
+            }],
+            rowLimit: 10000,
+            limit: 10000,
+            dimensions: [],
+            timeDimensions: [],
+            queryType: 'regularQuery'
+          }
+        ]);
+      }
+    );
+  });
+
   test('normalize queryRewrite limit', async () => {
     const { app } = createApiGateway(
       new AdapterApiMock(),