Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(Trello Node): Use body for POST requests #10189

Merged
merged 1 commit into from
Jul 25, 2024

Conversation

feelgood-interface
Copy link
Contributor

Summary

Trello API also supports replacing query parameters with a JSON request body instead. Using the request body avoids too long URLs which could produce 413 responses (Content Too Large).

@feelgood-interface feelgood-interface changed the title fix(LinkedIn Node): Use body for POST requests fix(Trello Node): Use body for POST requests Jul 25, 2024
@n8n-assistant n8n-assistant bot added community Authored by a community member node/improvement New feature or request in linear Issue or PR has been created in Linear for internal review labels Jul 25, 2024
Copy link
Member

@Joffcom Joffcom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes sense to me and the change looks good, Thanks.

@Joffcom Joffcom merged commit 7775d50 into n8n-io:master Jul 25, 2024
10 checks passed
@github-actions github-actions bot mentioned this pull request Jul 31, 2024
@janober
Copy link
Member

janober commented Jul 31, 2024

Got released with n8n@1.53.0

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
community Authored by a community member in linear Issue or PR has been created in Linear for internal review node/improvement New feature or request Released
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants