Skip to content

Commit b94cf1d

Browse files
committed
fix(ses): stop coercing switch string 'false' to true in create_configuration_set
Boolean('false') evaluates to true, so turning off the reputationMetricsEnabled or sendingEnabled switch sent the opposite of the user's choice to SES. Match the established === 'true' string comparison pattern used elsewhere in the codebase.
1 parent 86594a7 commit b94cf1d

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

apps/sim/blocks/blocks/ses.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -593,8 +593,10 @@ export const SESBlock: BlockConfig<ToolResponse> = {
593593
if (rest.tlsPolicy) result.tlsPolicy = rest.tlsPolicy
594594
if (rest.sendingPoolName) result.sendingPoolName = rest.sendingPoolName
595595
if (rest.reputationMetricsEnabled != null)
596-
result.reputationMetricsEnabled = Boolean(rest.reputationMetricsEnabled)
597-
if (rest.sendingEnabled != null) result.sendingEnabled = Boolean(rest.sendingEnabled)
596+
result.reputationMetricsEnabled =
597+
rest.reputationMetricsEnabled === true || rest.reputationMetricsEnabled === 'true'
598+
if (rest.sendingEnabled != null)
599+
result.sendingEnabled = rest.sendingEnabled === true || rest.sendingEnabled === 'true'
598600
if (rest.suppressedReasons) result.suppressedReasons = rest.suppressedReasons
599601
if (rest.tags) result.tags = rest.tags
600602
break

0 commit comments

Comments
 (0)