Just as a note. the function in ftp.php listDirectoryContents also needs to be changed.

protected function listDirectoryContents($directory, $recursive = true)
{
    $listing = ftp_rawlist($this->getConnection(), '-lna ' . $directory, $recursive);

    if ($listing === false) {
        return array();
    }

    if (substr($listing[0], 0, 5) === "ftpd:") {
        return array();
    }

    return $this->normalizeListing($listing, $directory);
}

@kristianedlund Hi, I'm not familiar with synology NAS. I'm assuming this is a windows ftp server, correct?

@FrenkyNet No the NAS is running on a unix system and most of the answers comes out as correct unix answers. Such as the stat command when the file is found. However, it has an extra line when the file is not found.

I have a local copy working with the synology server, and I am just working towards making it a unit tested version before submitting a pull request for it.

@kristianedlund awesome, looking forward to the PR! Would maybe even make sense to extend the Ftp adapter rather than handling different cases in one.

@FrenkyNet That is up to you. I can do either one I think.
From a user perspective I can see both pros and cons.
The pro is that you get leaner code for you specific ftp
The con is that you have to know which FTP server you are connecting to.

But from the whole issue I can see around windows based I think you might end up with a whole lot of FTP adapters. On the other hand it is not a bigger issue than it can be refactored later on.

Just let me know which one you prefer and I will aim for that.