Add support for files in ftp servers

[GtheSheep]

Let me know if you think this should be merged with the function for SFTP files?

Also there's a bit of a temp solution to just hide credentials from the path as mentioned in #22 - happy to switch this to something better then just replacing with empty strings if you have a preference for how it should work?

[menzenski]

Let me know if you think this should be merged with the function for SFTP files?

IMO this is fine to keep separate.

Also there's a bit of a temp solution to just hide credentials from the path as mentioned in #22 - happy to switch this to something better then just replacing with empty strings if you have a preference for how it should work?

Thanks for adding this masking - this is maybe a nit, but I wonder if it'd be preferable to replace with asterisks: like re.sub('sftp://.*?@', "********", path, flags=re.DOTALL) - maybe that'd be more understandable to consumers?

[GtheSheep]

@menzenski - Ah yeah good call, I think that makes it clearer that masking has occurred rather than an error in reading credentials, will update