Skip to content
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

Error in file packages/nodes-base/nodes/Box/Box.node.ts causing Box node to fail with error "responseData.push is not a function" #4140

Closed
jjboardman opened this issue Sep 19, 2022 · 2 comments · Fixed by #4147
Assignees
Labels
node/issue Issue with a node

Comments

@jjboardman
Copy link

Describe the bug
Believe there is an error in [packages/nodes-base/nodes/Box/Box.node.ts] file line: 535 causing Box node to fail on folder creation (and possibly others?) on v0.193.0+.
The error n8n outputs is responseData.push is not a function. I believe responseData.push(...executionData); should be returnData.push(...executionData); if it is supposed to be there at all.

To Reproduce
Update to v0.193.0+ and attempt to create a folder using the box node.

Expected behavior
For there to be no error thrown .

Environment:

  • OS: Ubuntu 22.04
  • n8n Version 0.193.0+
  • Node.js Version 14.16.0
  • Database system Postgres
  • Operation mode queue
@Joffcom Joffcom added the node/issue Issue with a node label Sep 20, 2022
@Joffcom Joffcom self-assigned this Sep 20, 2022
@Joffcom
Copy link
Member

Joffcom commented Sep 20, 2022

Hey @jjboardman,

Thanks for reporting this one, I will take a look shortly 👍🏻

@Joffcom
Copy link
Member

Joffcom commented Sep 20, 2022

Hey @jjboardman,

I managed to reproduce the issue and have fixed it in #4147, It looks like while the node is throwing an error it is still working in the background. As a temporary solution until the fix is released you could use the continue on error option or ideally downgrade to 0.192.x.

Thanks again for reporting this one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
node/issue Issue with a node
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants