-
Notifications
You must be signed in to change notification settings - Fork 6.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HTTP node Max Pages expression ignored #9597
Comments
Hey @NilsW86, Thanks for the report I have managed to reproduce this and have created |
Hi @NilsW86 this has been fixed in the linked PR and will be released next week! |
Thanks! |
Fix got released with |
Hi @Joffcom @elsmr and @janober , I think the bug is still present although slightly different. I am now on the cloud version 1.55.3. I have this test case: This data is pinned to the manual node: My HTTP node works great when using "Max pages" as "Fixed" and input 3. It also works with a expression of {{ 3 }} (which is did not use to). However, it does not work with the expression {{ $json.filter.pageLimit }}. I am 100% sure that the data is available, I can even see that n8n can identify it: But when executing with {{ $json.filter.pageLimit }} instead of {{ 3 }}, I get an error. It says "No execution data available" even though it is: Meaning that it can still not handle a proper input parameter. I have also tried adding a set node before with a different parameter, not working either. Can you confirm that you can reproduce or so you need more information? Thanks! |
Fix got released with |
Bug Description
When using a JS expression in the "Max Pages" parameter (in Pagination/Limit Pages Fetched) in a HTTP Request node the limit is ignored. Any expression including {{ 2 }} can be used and it is still ignored so it is not due to the data not being available. Others have been experiencing the same thing:
https://community.n8n.io/t/need-some-help-with-http-request-node-pagination-using-max-pages-expression/39449
To Reproduce
Expected behavior
The expected behavior would be that an expression with {{ 2 }} would limit the pages fetched like a Fixed 2 would.
Operating System
n8n cloud
n8n Version
1.42.1
Node.js Version
The one n8n cloud is using
Database
SQLite (default)
Execution mode
main (default)
The text was updated successfully, but these errors were encountered: