FTP over TLS planned or workaround known?

Questions
1

Hi,

i just had the following error:

image
image1006×464 108 KB

I checked the code of the node and there is no support for TLS currently right? Is this planned at the moment or do you see any other workaround to use FTP with TLS?

As far as iI could see, you are using https://www.npmjs.com/package/promise-ftp which has the following config.object:

" * secureOptions <object>: Additional options to be passed to tls.connect(). Default: (none)"

So if you tell me it is not planned yet, I will test creating a new FTP beta node in my instance and will try to test this and share the result in Github of course if it works :slight_smile: .

Thank you very much for your time reading this and have a great evening
Benjamin

2 Likes
2

are you talking about SFTP?
image

3

Hi @RedPacketSec ,

no unfortunately not, I tried that too but FTP over TLS is also called FTPS. This is an extension to the File Transfer Protocol (FTP) that adds support for the Transport Layer Security (TLS).

BR
Benji

1 Like
4

FTPS support is not a bad idea, I prefer SFTP as it is easier to use and only one version with one port which is handy.

Would be nice to see both implicit and explicit FTPS added with certificate authentication for those that need it.

If you need help extending the current FTP node let me know, I spent a few years dealing with file transfer protocols and the security around them :slightly_smiling_face:

4 Likes
5

It’s not planned. However, you can test it locally, passing the extra parameter secure, as you mentioned. If that works, then we can expose it in the credentials. The only concern is that according to the parameter description, that parameter seems to be deprecated?

image
image1426×422 59.2 KB

2 Likes
6

That will be for implicit FTPS (FTPS using port 990), explicit FTPS is still all good.

2 Likes
7

Hi @RicardoE105,

Can we bring this to life again? I have to FTPS a file to a server every day, so adding secure to the credentials would make my life a lot easier.

Luke

8

@Benjamin_Exner did you manage to come up with a workaround for this? I have the same problem now.

9

Hi @renetheastronaut ,

unfortunately not, I wanted to change the code by adding some additional params to the node but between Work/Projects and Family/Friends did not find the time yet :frowning: . But it is still on my list of things I want to do as soon as I have time :sweat_smile:

1 Like
10

Hi @Benjamin_Exner,
totally understand that. Would be nice if you could keep us updated here. Thanks a lot :blush:

11

I’ve got FTPS working, and a pull request here. Unfortunately as described in the pull request, it depends on promise-ftp updating the version of @icetee/ftp that it uses. It’s a simple manual fix, but unless either promise-ftp updates, or the n8n team decides to fork and publish their own version of promise-ftp, it’s not something that can get merged quite yet.

2 Likes