🚸 (webhook) Improve header and query params parsing

apps/viewer/src/pages/api/typebots/[typebotId]/blocks/[blockId]/executeWebhook.ts

@@ -1,5 +1,4 @@
 import {
-  KeyValue,
   PublicTypebot,
   ResultValues,
   Typebot,
@@ -28,6 +27,7 @@ import {
 } from '@typebot.io/schemas/features/blocks/integrations/webhook/constants'
 import { getBlockById } from '@typebot.io/lib/getBlockById'
 import {
+  convertKeyValueTableToObject,
   longReqTimeoutWhitelist,
   longRequestTimeout,
   responseDefaultTimeout,
@@ -280,20 +280,6 @@ const getBodyContent =
       : body ?? undefined
   }
 
-const convertKeyValueTableToObject = (
-  keyValues: KeyValue[] | undefined,
-  variables: Variable[]
-) => {
-  if (!keyValues) return
-  return keyValues.reduce((object, item) => {
-    if (!item.key) return {}
-    return {
-      ...object,
-      [item.key]: parseVariables(variables)(item.value ?? ''),
-    }
-  }, {})
-}
-
 // eslint-disable-next-line @typescript-eslint/no-explicit-any
 const safeJsonParse = (json: string): { data: any; isJson: boolean } => {
   try {

packages/bot-engine/blocks/integrations/webhook/executeWebhookBlock.ts

@@ -265,16 +265,18 @@ const getBodyContent = async ({
     : body ?? undefined
 }
 
-const convertKeyValueTableToObject = (
+export const convertKeyValueTableToObject = (
   keyValues: KeyValue[] | undefined,
   variables: Variable[]
 ) => {
   if (!keyValues) return
   return keyValues.reduce((object, item) => {
-    if (!item.key) return {}
+    const key = parseVariables(variables)(item.key)
+    const value = parseVariables(variables)(item.value)
+    if (isEmpty(key) || isEmpty(value)) return object
     return {
       ...object,
-      [item.key]: parseVariables(variables)(item.value ?? ''),
+      [key]: value,
     }
   }, {})
 }