Ftp connection import support.

CommentFileSizeAuthor
#5 feeds-ftp-2203927-5.patch847 bytesklausi
ftp.patch1.15 KBTrasherk
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Status: Needs review » Needs work

The last submitted patch, ftp.patch, failed testing.

The last submitted patch, ftp.patch, failed testing.

twistor’s picture

twistor CreditAttribution: twistor commented
Version: 7.x-2.0-alpha8 » 7.x-2.x-dev
Component: Feeds Import (feature) » Code
JKingsnorth’s picture

JKingsnorth CreditAttribution: JKingsnorth commented
Title: Ftp connection » Fetch by FTP connection

Looking at the file locations, these have just been added into the 'HTTP' request / fetcher options. Although this might work it doesn't make much sense since FTP is a different protocol. Wouldn't it be better to create a new FTP-specific one?

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
CreditAttribution: klausi at jobiqo - job board technology commented
Status: Needs work » Needs review
FileSize
feeds-ftp-2203927-5.patch847 bytes

Since this super little change is working so well I'm posting a rerolled patch. It does not matter that the other Curl/fetcher options don't make sense. FTP is already allowed as URL in the UI otherwise.