NETOBSERV-1958: fix CSV export (#643)

Do not stringify undefined fields as they end up == "undefined"
netobserv · Nov 13, 2024 · f8abad2 · f8abad2
1 parent 5e50466
commit f8abad2
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/web/src/model/__tests__/export-query.spec.ts b/web/src/model/__tests__/export-query.spec.ts
@@ -5,6 +5,7 @@ describe('buildExportQuery', () => {
     const query = buildExportQuery({
       filters: 'SrcK8S_Name%3Dtest1%2Ctest2',
       recordType: 'flowLog',
+      namespace: undefined,
       dataSource: 'auto',
       packetLoss: 'all',
       limit: 500,

diff --git a/web/src/model/export-query.ts b/web/src/model/export-query.ts
@@ -1,3 +1,4 @@
+import * as _ from 'lodash';
 import { FlowQuery } from './flow-query';
 
 export const buildExportQuery = (flowQuery: FlowQuery, columns?: string[]) => {
@@ -11,5 +12,6 @@ export const buildExportQuery = (flowQuery: FlowQuery, columns?: string[]) => {
   if (columns) {
     query.columns = String(columns);
   }
-  return new URLSearchParams(query).toString();
+  const omitEmpty = _.omitBy(query, a => a === undefined);
+  return new URLSearchParams(omitEmpty).toString();
 };