-
-
Notifications
You must be signed in to change notification settings - Fork 4.7k
Fix for Win Clients sometimes not connecting #3370
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
Conversation
Fix for Win Clients sometimes not connecting
|
So with "Win Client" you refer to Windows Explorer, correct? |
|
Yes, I am refering to Windows Explorer. |
|
@michag86 Could you try this one for your windows clients? |
|
Works 👍 |
| <IfModule mod_rewrite.c> | ||
| RewriteEngine on | ||
| RewriteCond %{HTTP_USER_AGENT} DavClnt | ||
| RewriteRule ^$ /remote.php/webdav/ [L,R=302] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This only works for the root directory and if the RewriteBase is set. Please add this to
Line 448 in d548329
| $content .= "\n RewriteRule ^core/js/oc.js$ index.php [PT,E=PATH_INFO:$1]"; |
/ before remote.php to be compatible with sub folders. Then this is good to go in
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In my case RewriteBase was not in use and ist works.
And it's only required for the root directory. If Nextcloud ist installed in a sub directory, the problem does not occour.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And it's only required for the root directory. If Nextcloud ist installed in a sub directory, the problem does not occour.
But we would redirect to /remote.php/webdav/ instead of ´/sub/remote.php/webdav/`... even if it is never requested like this it could cause huge headaches.
@LukasReschke @nickvergessen Opinions of introducing a rewrite condition that works and fixes a problem if installed in root, but creates a redirect to "outside of nextcloud" for the sub directory install, but there it is never requested by Windows anyways. (I tested this and it works)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If Nextcloud ist installed in a sub directory, the "RewriteRule ^$" in the .htaccess of the sub directory will do nothing. Or am I wrong?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If Nextcloud ist installed in a sub directory, the "RewriteRule ^$" in the .htaccess of the sub directory will do nothing. Or am I wrong?
It does, because it get's only the stuff handed in within it's subdirectory. And then it redirect to the root (because of the leading slash in /remote.php/dav). Therefore I tested this with windows and it doesn't request the subdirectory.
|
@rullzer @nickvergessen @icewind1991 @schiessle @LukasReschke Please review this one |
@LukasReschke I would merge this one, because @michag86 also tested this. |
|
i am sorry but both fixes (.htaccess and lib/base.php) are not working for me. since OC 11.0.1 i got the issue that Windows Explorer asks 3 times for a username and Password
|
|
401 means unauthorized. It doesnt recognize your password. However, not related to this. Open a new issue please, however I do not think it is an issue. Also this is for nextcloud, I will assume that OC was just a typo. If your issue is really an owncloud issue, this is clearly the wrong place. |
|
i am sorry, OC was a typo. |
Fix for Win Clients sometimes not connecting We discovered this during investigation of Issue #2887