-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
FTP only works with subdirectories #17
Comments
Hi, I've the same issue with the 10.0.2 owncloud release. I've tried in a docker container too but it's the same problem. A part of the response : "statusMessage":"League\Flysystem\FileNotFoundException: File not found at path: " @owncloud-bot @PVince81 @DeepDiver1975 Have you observed this trouble? |
Got exact same:
|
Hi @PVince81 @DeepDiver1975 or someone else, I've tried with the 10.0.3 owncloud release and always the same issue, FTP connection doesn't work at all. It's a problem because I can't update my Owncloud because it's an important feature for my institution. |
Same for me. FTP work only on subdirectory not on root. On root directory no connection at all to server. |
Does it work for everyone on subdirectories ? If yes, maybe it's just a simple path computation bug which would be good news and easy to fix. |
Can you try adding a "/" instead of subdirectory in the configuration instead of leaving it empty, in case that helps ? |
Hi, indeed, it works only with subdirectories but not on root, even with adding /. Thanks for your replies!! |
Me too, even with "/", no connection to root directory. |
Is there a docker instance with the same FTP server software that could be used to reproduce this quickly ? (I don't have an FTP instance locally for trying this) The automated tests are currently running against vsftpd (https://github.com/owncloud/files_external_ftp/blob/master/.travis.yml#L28). Can you guys post what FTP server software you are using ? |
Hi, I use a proftpd server. Maybe you can test with this public FTP server : host : speedtest.tele2.net Thank for you working on it. |
Hi @PVince81, any progress on it? |
Can you guys try replacing It seems the slash might be doubled here: https://github.com/owncloud/files_external_ftp/blob/master/lib/Storage/FTP.php#L72 and maybe ProFTP doesn't like it ? (I don't have time to debug this myself right now, sorry) |
Even with "" instead of "/" not work for me. |
Not good for me too :/ |
found a docker here: https://hub.docker.com/r/aexea/proftpd/ |
I've also encountered this problem: Using this docker to verify:
With And the exception in the json reply:
With a subdirectory previously created on the ftp it works: Json answer:
cc @PVince81 - let me know if you need further details on this |
Is there any update on the ftp sub dir issue ? |
The only thing I can say for now is that it might not be possible to fetch the required information from the root folder of the ftp server. I don't know for sure if it's a bug in the flysystem, or we'll have to patch it anyway because we can't get the information for the root folder. |
A little bit of random debugging from me. I tried all possible combinations ( "", "/", / , , ./ , /.. , etc. Νothing works and i get a monotonous "Message":"File not found at path: " Things got a bit out of the ordinary when I opted for ../ as a root folder and then I got the expected error "message":"Exception: {"Exception":"LogicException","Message":"Path is outside of the defined root, path: [..]" There might be a possibility that the system sends the correct path but the ftp server is messing up with the reply, or reply is not understood correctly. |
@jvillafanez can we use a virtual entry for root then ? |
we can fake the information based on the contents of the folder, but it might be expensive if there is a lot of content. Not sure if there is any other alternative. |
@PVince81 @jvillafanez I'm not sure i understand the notion of virtual entry in this context. I cannot thing of a virtual anything when it comes to / directory on ftp server. BTW tests are performed on vsftpd on Centos 7. |
the main problem as I understand is that a FTP server does not return any metadata for the root entry. So we don't have metadata like "folder size", "mtime", etc, so we cannot represent that node inside of ownCloud with real information. The idea above is that instead of asking the FTP server, ownCloud populates its internal root node information with computed information. The mtime is important to be able to detect changes that happened remotely. |
I think we'll also need to compute the mtime based on the contents. If this the case, this will be expensive because we'll have to scan the contents to know if something has changed. |
but only for the root dir - or? |
in this case "a bit expensive" is likely more desirable than "not working at all" I guess |
Any updates on this? I have some legacy devices running FTP and I'd like to have top / directory visible |
still not working |
For me this worked. I just used "/tmp/.." as directory. |
Installed from Marketplace files_external_ftp app.
Either in user or admin Storage area, I try to define an external FTP source. The status icon shows red.
In attempt to debug, I started a tshark capture and no traffic is generated from the host towards the FTP server.
Is there a prerequisite I'm missing? PHP is 7.1.5 built from source with --enable-ftp ticked. phpinfo indicates ftp and ftps streams are enabled. Setting owncloud logging to info,warnings,errors and fatal issues shows nothing in the logs.
System Status
installed true
maintenance false
needsDbUpgrade false
version 10.0.1.5
versionstring 10.0.1
edition Community
productname ownCloud
Debugging pointers appreciated.
The text was updated successfully, but these errors were encountered: