SK-704 added encodeURI option default true in get

skyflowapi · Oct 20, 2023 · 0be8bf9 · 0be8bf9
1 parent fe8f842
commit 0be8bf9
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/src/vault-api/core/Reveal.ts b/src/vault-api/core/Reveal.ts
@@ -65,14 +65,14 @@ const getSkyflowIdRecordsFromVault = (
     paramList += `skyflow_ids=${skyflowId}&`;
   });
 
-  if(options && Object.prototype.hasOwnProperty.call(options, 'encodeURI') && options?.encodeURI === true) {
+  if (options && Object.prototype.hasOwnProperty.call(options, 'encodeURI') && options?.encodeURI === false){
     skyflowIdRecord.columnValues?.forEach((column) => {
-      var encode_column_value = encodeURIComponent(column)
-      paramList += `column_name=${skyflowIdRecord.columnName}&column_values=${encode_column_value}&`;
+      paramList += `column_name=${skyflowIdRecord.columnName}&column_values=${column}&`;
     });
   } else {
     skyflowIdRecord.columnValues?.forEach((column) => {
-      paramList += `column_name=${skyflowIdRecord.columnName}&column_values=${column}&`;
+      var encode_column_value = encodeURIComponent(column)
+      paramList += `column_name=${skyflowIdRecord.columnName}&column_values=${encode_column_value}&`;
     });
   }