Skip to content

fix(http): expose structured vars in HTTP body selector#34185

Merged
crazywoola merged 2 commits intolanggenius:mainfrom
owldev127:fix/http-node-body-structured-vars
Mar 31, 2026
Merged

fix(http): expose structured vars in HTTP body selector#34185
crazywoola merged 2 commits intolanggenius:mainfrom
owldev127:fix/http-node-body-structured-vars

Conversation

@owldev127
Copy link
Copy Markdown
Contributor

@owldev127 owldev127 commented Mar 27, 2026

Important

  1. Make sure you have read our contribution guidelines
  2. Ensure there is an associated issue and you have been assigned to it
  3. Use the correct syntax to link this PR: Fixes #<issue number>.

Summary

Fixes #34153

Problem

Users could not select non-string structured variables in the HTTP node body editor, even though those values were valid workflow outputs and expected to work in request bodies. This forced manual serialization as a workaround.

The frontend HTTP body editor filtered available variables too aggressively and excluded VarType.object and VarType.arrayObject before the selector tree was built, so those variables never appeared in the picker.

Solution

Allow object and array[object] variables in the HTTP body selector and add a regression test.

Screenshots

Before After
image image

Checklist

  • This change requires a documentation update, included: Dify Document
  • I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
  • I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
  • I've updated the documentation accordingly.
  • I ran make lint and make type-check (backend) and cd web && npx lint-staged (frontend) to appease the lint gods

@dosubot dosubot Bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Mar 27, 2026
@github-actions github-actions Bot added the web This relates to changes on the web. label Mar 27, 2026
@owldev127 owldev127 force-pushed the fix/http-node-body-structured-vars branch from 7b83437 to d3edec8 Compare March 27, 2026 08:33
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Mar 31, 2026
@crazywoola crazywoola enabled auto-merge March 31, 2026 02:01
@crazywoola crazywoola disabled auto-merge March 31, 2026 02:20
@crazywoola crazywoola merged commit 2c2cc72 into langgenius:main Mar 31, 2026
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files. web This relates to changes on the web.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HTTP Node does not expose non-string variables (array/object) in template selector

2 participants