<#5352 [Core feature] Support ftp for FlyteFile> I...
# flytekit
a
#5352 [Core feature] Support ftp for FlyteFile Issue created by ggydush Motivation: Why do you think this is important? Various bioinformatic reference files still use ftp :( It would be awesome if FlyteFile supported ftp URLs for downloading. Goal: What should the final outcome look like, ideally? Allow using ftp URLs with FlyteFile Describe alternatives you've considered Use curl or wget, which isn't as clean. Propose: Link/Inline OR Additional context
Copy code
from flytekit.types.file import FlyteFile


ff = FlyteFile.from_source("<ftp://ftp.ncbi.nlm.nih.gov/pub/taxonomy/new_taxdump/new_taxdump.tar.gz>")
ff.download()
Currently this produces:
Copy code
╰────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────╯
FlyteAssertion: USER:AssertionError: error=Failed to get data from <ftp://ncbi.nlm.nih.gov/pub/taxonomy/new_taxdump/new_taxdump.tar.gz> to 
/var/folders/d1/n1tj86t11pj_drgfg7_br5n00000gn/T/flyte-3z705rwd/sandbox/local_flytekit/b972901582e5f3d4fd7d7b806a53f84f/new_taxdump.tar.gz (recursive=False).

Original exception: FTPFileSystem.__init__() missing 1 required positional argument: 'host'
Are you sure this issue hasn't been raised already? ☑︎ Yes Have you read the Code of Conduct? ☑︎ Yes flyteorg/flyte