-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
NC12 Symlink and "ResourceLocator can not find a web root" #5057
Comments
I can confirm this issue. |
This seems to be biting the Nextcloud snap, as well. |
@oparoz can we get this sooner than 13? Otherwise there's no way to update snap users (since they'll need to jump from 11->13, right?). |
Ok I shouldn't be changing the labels here but yeah, it'd be nice to have this fixed before Nextcloud 12 so the users of the Box and other Snap users can use Nextcloud 12. May I ask why this was set to enhancement instead of a regression and bug and why only for 13, @oparoz |
The same issue was found here: #4857. I closed it as this one is accepted as bug already 💯 |
I set it as an enhancement, thinking that this was not an officially supported way of using config.php, but let's try to fix the new CSS compiler. |
Adding my 2 cents, I have only one symlink at the root of the installation:
And I have the bad layout on Files and these errors in the logs
|
In my case, I have no symlinks...... |
Another use case is the following: let /var/www/nextcloud be a symlink to some external folder. In that case the config.php and apache/ngnix config is "default", but the underlying type is not a folder, but a link. This is what I did and leads to exact the same issue. |
In the meantime, is there any workaround, like generate CSS files manually? |
In my case, I am also finding this in my log:
I forgot to mention that I am using the 12.0.0 stable version that is being announces on the official webpage as the version to use. So I would think that such a visible bug should get a swift attention.. |
Please, let me know what information can I provide to help to fix this problem. |
I apologize, I forgot that I had the entire Fortunately, getting rid of the link fixed the issue for me. |
In case you want to help, I think the change was introduced with #3619. |
Thank you for the link @derkostka ! And I apologize again for the misinformation. |
I have the same issue, that's my current workaround: https://gist.github.com/sileht/5753118ee048352778c2a0c5b23ea47d |
The solution posted by sileht seems to be correct. Could someone merge it? |
@sileht I can't judge the code but if it works, pls consider creating a PR ;-) |
I just write a workaround, I don't known if that the "good" solution and I have no time a babysit a PR for this. |
Fixed in 12.0.1 and 13. |
@sileht @derkostka thanks a lot for making this happen ;-) It might be a small patch but we have over 1500 people with a Nextcloud Box at home who can now get 12.0.1 and that's ignoring the many thousands who use the Snap on a normal Ubuntu or other Linux system! |
Happens for me for 13.0.2 |
@Temtaime it is supposed to be fix, could you open a new issue? :) |
Steps to reproduce
Write symlink path in config.php
( i have symlink - /www/nextcloud to /srv/www/htdocs/nextcloud)
Error in log
Debug files_texteditor Could not find resource file "/css/files_texteditor/merged.css" 2017-05-23 11:45:07
Error lib ResourceLocator can not find a web root (root: /www/nextcloud/apps/files_texteditor, file: css/files_texteditor/merged.css, webRoot: , throw: true) 2017-05-23 11:45:07
Debug files_sharing Could not find resource file "/css/files_sharing/mergedAdditionalStyles.css" 2017-05-23 11:45:07
Error lib ResourceLocator can not find a web root (root: /www/nextcloud/apps/files_sharing, file: css/files_sharing/mergedAdditionalStyles.css, webRoot: , throw: true) 2017-05-23 11:45:07
Debug files Could not find resource file "/css/files/merged.css" 2017-05-23 11:45:07
Error lib ResourceLocator can not find a web root (root: /www/nextcloud/apps/files, file: css/files/merged.css, webRoot: , throw: true)
CSSResourceLocator.php
append function get param
put to file $root,$file
...
/srv/www/htdocs/nextcloud css/core/server.css
/srv/www/htdocs/nextcloud css/core/share.css
/www/nextcloud/apps/files css/files/merged.css
/www/nextcloud/apps/files_sharing css/files_sharing/mergedAdditionalStyles.css
/www/nextcloud/apps/files_texteditor css/files_texteditor/merged.css
/srv/www/htdocs/nextcloud css/core/systemtags.css
/srv/www/htdocs/nextcloud css/core/server.css
/srv/www/htdocs/nextcloud css/core/share.css
...
The text was updated successfully, but these errors were encountered: