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

Move files expose complete folder structure on public links #2334

Closed
schiessle opened this issue Nov 25, 2016 · 3 comments
Closed

Move files expose complete folder structure on public links #2334

schiessle opened this issue Nov 25, 2016 · 3 comments

Comments

@schiessle
Copy link
Member

  • Share a folder with some files as public link with edit permissions
  • open the public link
  • the "three dot menu" of the files will show the "move" operation
  • click on it

move2

Result:

You see the root of the users files folder. You can move the files to any location and look at the complete folder structure of the user

Expected:

Only show folders within the shared folder

cc @icewind1991

@LukasReschke
Copy link
Member

I think that is because the WebDAV call is performed with the wrong credentials. Can you try in a private tab?

So there is no actual real loss of information but simply the credentials of the logged-in users used. At least from how I understand the code.

@LukasReschke
Copy link
Member

Yep. Move seems to connect against /remote.php/webdav instead /public.php/webdav

cc @icewind1991 Can you take a look? THX :)

@schiessle
Copy link
Member Author

yes, in a private tab we don't expose the whole directory structure but the dialog is still broken. It shows a spinner and "Problem loading page, reloading in 5 seconds"

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