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

Increase Max Payload #37

Closed
BenJamesAndo opened this issue May 2, 2023 · 7 comments
Closed

Increase Max Payload #37

BenJamesAndo opened this issue May 2, 2023 · 7 comments

Comments

@BenJamesAndo
Copy link

BenJamesAndo commented May 2, 2023

n8n by default seems to only allow files of 16MB or smaller to be processed. As per the recent update where the /backup folder is recognised I'd like to be able to process some files that are larger than 16MB, as are most backup files.

Is it possible to add an addon configuration to increase the max size? e.g. 2048MB. The variable is N8N_PAYLOAD_SIZE_MAX as per n8n environment variables reference guide

@Rbillon59
Copy link
Owner

Hi @BenJamesAndo !

What is your use case here ? I tried to upload a backup file (190MB) to a FTP using n8n and it was successful.
image

Also, keep in mind, if working with binary data, they will be loaded in JVM, so you need as more heap size as the file size

@BenJamesAndo
Copy link
Author

BenJamesAndo commented May 21, 2023

Hi @Rbillon59 I probably need to learn n8n better. Because I can't figure out how to upload to FTP without first using a 'Read Binary Files' node. If I do it straight from a local file trigger to FTP it uploads a file with the same name, but it's much smaller. I think the file it creates is just a file with the 'path' as the contents, rather than the actual contents of the file.

image
image

If I do use a binary file node then I get the max payload error.

@Rbillon59
Copy link
Owner

Thank you for your explanation. You're totally right and I was wrong. You've not to learn how to use n8n, I have haha.

I will think about the implementation. You're not the first to ask for a way to change an env var. Maybe I can create an array of key value pairs to pass to the configuration so any env var could be changed in the configuration without having to create a new release of the addon each time.

I'll keep you up

@BenJamesAndo
Copy link
Author

Hey that sounds like a cool idea! Because I was actually thinking having control over the WEBHOOK_TUNNEL_URL & EXECUTIONS_PROCESS env variable would be good. Would actually be really nice if all HA addons allowed for complete control over env variables. Hopefully you figure out a way to pull it off.

@Rbillon59
Copy link
Owner

Could you please update the addon and try what you want ? Let me know if you face any issue.

Thanks

@BenJamesAndo
Copy link
Author

I've tested a bunch of variables and they work great. Thanks! It's a very nice implementation of the feature.

@Rbillon59
Copy link
Owner

Awesome ! Closing the issue, enjoy !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants