vyuldashev / flysystem-curlftp

Flysystem Adapter for the FTP with cURL implementation
MIT License
36 stars 24 forks source link

Adds config option to disable LIST command arguments #35

Closed marvinschroeder closed 2 years ago

marvinschroeder commented 2 years ago

This pull requests adds a config option to disable the LIST command arguments. It is useable for newer FTP servers, like FileZilla Server versions > 1.1.0, were the FTP command LIST is no longer supporting arguments like "-A" or "-aln".

The additional commit adds the root folder to the writeStream and readStream methods, since CURLOPT_URL needs an absolute URL and the CWD command inside the setConnectionRoot method has no effect for the two operations.

vyuldashev commented 2 years ago

Tests are failing. Could you please take a look?

marvinschroeder commented 2 years ago

I just traveled back in time down to version 2.1.0 and tests were already failing without my additions. Maybe docker images broken or updated since first written tests?