Skip to content
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

Cyberduck not listen files #1116

Closed
francoricci opened this issue Aug 31, 2018 · 6 comments
Closed

Cyberduck not listen files #1116

francoricci opened this issue Aug 31, 2018 · 6 comments

Comments

@francoricci
Copy link

Until version 6.2.2 cyberduck listen files inside leofs buckets and directories.
Latest version of cyberduck can list only directories but now files inside is accessible.

@mocchira mocchira self-assigned this Sep 4, 2018
@mocchira
Copy link
Member

mocchira commented Sep 4, 2018

@francoricci Thanks for reporting this issue. I've confirmed on the latest (6.7.1.28683). Since other s3 clients works fine, there might be something wrong specific to the cyberduck latest. I'll dive into the cyberduck code from now on. Until I find the fix, would you like to keep using 6.2.2 cyberduck or try dragondisk (http://www.s3-client.com/) out as a GUI browser?

@francoricci
Copy link
Author

I'll use the old version f CyberDuck until solution.
Thanks

@mocchira
Copy link
Member

mocchira commented Sep 6, 2018

It turns out that several comments I previously posted at here were wrong so I deleted those to keep comments clean.

@mocchira
Copy link
Member

mocchira commented Sep 6, 2018

The reason why the latest cyberduck can't display any files but display folders stored in LeoFS is that they've changed the API retrieving the object list from https://docs.aws.amazon.com/AmazonS3/latest/API/RESTBucketGET.html to https://docs.aws.amazon.com/AmazonS3/latest/API/RESTBucketGETVersion.html.

Since LeoFS has no yet supported GET Bucket Object versions API, any files have not been displayed on the latest cyberduck. At a first glance, the simple implementation which just returns the most recent objects can be implemented in quite less cost so that I'm going to implement it in that way and confirm if it works as expected.

mocchira added a commit to mocchira/leofs that referenced this issue Sep 6, 2018
@mocchira
Copy link
Member

mocchira commented Sep 6, 2018

@francoricci I've just sent PR #1118 to fix the problem. The next stable release 1.4.3 will include the fix so once 1.4.3 comes out and you finish to upgrade LeoFS, you will be able to use the latest cyberduck. Please wait for a while.

@francoricci
Copy link
Author

Great!
Thanks a lot, I'll wait for the next leofs release.

Franco

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants