Merge pull request #314 from lowcoder-org/fix-302-lamda-datasource-query-failed-to-run

FalkWolsky · web-flow · commit ab68b678b7d5 · 2023-07-29T22:03:14.000+02:00
Add Missing Existence Check For dynamicParamsConfig That Causes FireBase Conenctor Query Run Failure
diff --git a/server/node-service/src/services/plugin.ts b/server/node-service/src/services/plugin.ts
@@ -214,7 +214,7 @@ export async function runPluginQuery(
 
   //forward cookies
   context.forEach(({ key, value }) => {
-    if (key in dataSourceConfig.dynamicParamsConfig) {
+    if (dataSourceConfig.dynamicParamsConfig && key in dataSourceConfig.dynamicParamsConfig) {
       const valueKey = `${key}.value`;
       dataSourceConfig.dynamicParamsConfig[valueKey] = value[0].value
     }

Original file line number	Diff line number	Diff line change
`@@ -214,7 +214,7 @@ export async function runPluginQuery(`
`214`	`214`
`215`	`215`	`//forward cookies`
`216`	`216`	`context.forEach(({ key, value }) => {`
`217`		`- if (key in dataSourceConfig.dynamicParamsConfig) {`
	`217`	`+ if (dataSourceConfig.dynamicParamsConfig && key in dataSourceConfig.dynamicParamsConfig) {`
`218`	`218`	const valueKey = `${key}.value`;
`219`	`219`	`dataSourceConfig.dynamicParamsConfig[valueKey] = value[0].value`
`220`	`220`	`}`