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

Cadaver does not see remote shares #9149

Closed
PVince81 opened this issue Jun 23, 2014 · 5 comments · Fixed by #9150
Closed

Cadaver does not see remote shares #9149

PVince81 opened this issue Jun 23, 2014 · 5 comments · Fixed by #9150

Comments

@PVince81
Copy link
Contributor

Steps to reproduce

  1. Setup two servers "localserver" and "remoteserver".
  2. On "localserver" enable ext storage
  3. On "remoteserver" share a folder "testshare" with link and allow upload
  4. Mount "testshare" on "localserver" using server to server sharing
  5. Open cadaver with "http://localserver/remote.php/webdav"
  6. ls from the root

Expected result

ls should show "testshare" in the list.

Actual result

"testshare" does not appear.
Even worse, doing a mkdir testshare creates an empty folder "testshare" that appears in the data dir.

More info

This is with OC 7 git master 81ecbbb

Note that using WebDAV through the web UI or through KDE's Dolphin correctly displays that folder.

Local shares do not seem to be affected.

It seems that somehow the code that mounts remote shares is bypassed for some strange reasons when using Cadaver (cadaver-0.23.3-70.1.2.x86_64)

@icewind1991 @schiesbn

@PVince81
Copy link
Contributor Author

Setting to OC 7 CE as this could cause breakage in the file structure (CC @karlitschek @craigpg)

@PVince81 PVince81 added this to the ownCloud 7 CE milestone Jun 23, 2014
@PVince81
Copy link
Contributor Author

I'll try and debug to find what the difference is between the call from Dolphin and Cadaver.

@PVince81
Copy link
Contributor Author

It's very weird, it looks like user_id is not defined in the user session, so the sharing mount manager doesn't do anything. It almost looks like the session through cadaver is only half initialized. Still, we're able to see the other files.

@PVince81
Copy link
Contributor Author

It looks like cadaver is re-authing the session for every call where Dolphin seems to be reusing the session.
The issue happens when re-authing the session: the external mount manager calls setup in app.php which happens when loadApps is called. But that seems to happen too early, before the session is even initialized.

I'll try and move the external mount manager setup to be done during a setup hook.

@PVince81
Copy link
Contributor Author

Fix is here: #9150

@lock lock bot locked as resolved and limited conversation to collaborators Aug 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant