Elsa Workflows中Http Request活动处理表单数据的Bug解析与修复

问题背景

在Elsa Workflows工作流引擎中，Http Request活动是用于发送HTTP请求的核心组件。当需要发送"application/x-www-form-urlencoded"类型的内容时，系统使用FormUrlEncodedHttpContentFactory类来处理内容转换。然而，这个实现存在两个主要问题：

1. 当内容为字符串时，系统会先序列化再反序列化为字典，但字符串序列化后无法正确反序列化为字典结构
2. 当内容为ExpandoObject动态对象时，Convert.ChangeType方法无法处理这种类型转换

技术细节分析

FormUrlEncodedHttpContentFactory类的原始实现逻辑如下：

private static Dictionary<string, string> GetContentAsDictionary(object content) =>
    (content is string or JsonObject
        ? JsonSerializer.Deserialize<Dictionary<string, string>>(JsonSerializer.Serialize(content))
        : (Dictionary<string, string>)Convert.ChangeType(content, typeof(Dictionary<string, string>)))!;

这段代码存在几个技术问题：

1. 字符串处理不当：对字符串内容先序列化再反序列化的方式，会导致类似"key1=value1&key2=value2"这样的标准表单字符串被错误处理。字符串会被序列化为带引号的JSON字符串，然后无法正确反序列化为字典。

2. 动态对象支持不足：ExpandoObject是.NET中常用的动态类型，常用于JavaScript表达式的结果。原始代码尝试使用Convert.ChangeType进行转换，但这种方法不适用于动态对象。

3. 类型判断逻辑不完善：仅检查string和JsonObject两种类型，忽略了实际工作流中常见的其他数据类型。

解决方案

Elsa团队已经修复了这个问题，主要改进包括：

1. 直接处理字符串内容：对于字符串输入，不再进行序列化/反序列化操作，而是直接解析为字典。

2. 增强类型支持：增加了对IDictionary<string, string>、IDictionary<string, object>等常见字典类型的直接支持。

3. 改进动态对象处理：对于ExpandoObject等动态类型，采用更合理的转换方式。

实际应用建议

在使用Elsa Workflows的Http Request活动发送表单数据时，开发者可以：

1. JavaScript表达式：可以直接返回一个对象字面量，如{key1: "value1", key2: "value2"}，系统会自动正确处理。

2. Liquid模板：确保输出是标准的键值对格式，或者直接构造一个字典对象。

3. 其他表达式：可以直接返回.NET的Dictionary<string, string>或类似结构。

总结

这个修复显著提升了Elsa Workflows中表单数据处理功能的健壮性和易用性。开发者现在可以更灵活地使用各种数据格式来构造HTTP请求的表单内容，而无需担心底层转换问题。这也体现了Elsa团队对开发者体验的重视，通过不断改进核心组件来提供更流畅的工作流开发体验。