-
Notifications
You must be signed in to change notification settings - Fork 87
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
HomePage Loading Circle #820
Comments
Thanks for reporting 👍 Added the "needs info" label because without server and client logs (as described in the issue template) it's impossible to figure out whats wrong. |
Also getting this error after a clean install on ubuntu server.
Gone through and checked all libs and installation requirements are met and config is correct. |
Okay traced this and if no README.md is located in the directory you are in this loading circle will just sit there.
|
Mind to check your network tab and share the failing xhr request? |
404 is the correct response if there is no README.md found. Yeah it's a bad choice (because the webserver will log it). It works fine over here. 404 response, spinner is gone. |
The spinner is still there... |
The response for the failing xhr request is? |
404 !? Where is the property "data"? |
You disabled the text app? |
Anything in Actually logs, configuration and apps are already part of the issue template and I already added the "needs info" label earlier because those information are missing. You see there is a reason we ask that information. Any chance that your webserver is changing the response? The response should not be empty and have response data. |
Just did a tail -f /data/nextcloud.log and nothing new appears even after a force CTRL+F5 on the page |
Request for a folder without readme.md
Request for a folder with README.md
Is the text app disabled there are no such requests. If one needs a test instance to compare the requests / responses: https://try.nextcloud.com/. Looks like a configuration issue to me. We probably need more information to investigate further. |
Confirmed. If i disable text app it's ok! What config do you refer? I never configured nothing for the text app... |
I suspect that your webserver changes the response. The code does not check if the response is the expected response. It's assumed that the 404 response contains data. Should be possible to check this. But the actual issue is your configuration / webserver changing the 404 response. |
Ok...strange this didnt happen before the latest update. I'm running nextcloud docker image... |
Same issue here after upgrading to 18.0.4, running behind nginx "proxy" and a standard next cloud install on the other end. EDIT: Disabling the text app makes the spinner disappear. |
I ended up turning off rich workspace for now. |
I cannot reproduce this. Please try to provide the output of the requests that @kesselb provided. |
It's the result as @nunofmds got, 404 code with empty response, |
Can you run it with -v to get the http headers as well? |
|
Hi,
Removing those, which are no longer in the provided Nginx configuration fixes the Issue for me. Kind Regards |
Ok, yes that would explain it. Basically if the workspace returns a 404 because there is no readme file yet for that directory your nginx is trying to hand out an error page that does not exist anymore and therefore it redirects to the files app itself, which we cannot handle as the text app. Maybe we can add some hardening to not fail in such a redirect case. |
Nice find! I had that setting too and after removing it, the spinning circle is gone. |
As @kesselb suggested it might worth to move away from returning a 404 here and rather use a 2xx status code or even get rid of the /workspace endpoint and use the custom dav property we already have for mobile integration instead. |
Wow, amazing that this solved it for me! How did you find it? Thanks a lot |
For thoose that use linuxserver/nextcloud docker image the nginx config file is on /config/nginx/site-confs/default edit the file and comment error_page 403 /core/templates/403.php; restart the docker and should be fixed... worked for me as well... Thanks! |
Let's keep that open to track the improvements suggested from #820 (comment) |
Had the same issue. Ubuntu 20.04, nginx. Disabling Rich Workspaces also gets rid of the spinning wheel. But neither of these is a fix. Creating a Readme.md file in a directory ends the spinning wheel and loads the "Add notes, lists or links..." dialogue. However, commenting out the following lines in /etc/nginx/conf.d/nextcloud.conf
Did the trick. |
linuxserver.io team member here Those were removed from our default site conf well over a year ago, for Nextcloud v15: linuxserver/docker-nextcloud@39fd8e3#diff-cab6a3a029fd4edc2e44acec10375bdcL44-L45 Our changelog includes notes about existing users having to delete the default site conf and restart the container to update whenever there is a change such as this one: |
Same issue with apache server from https://github.com/nextcloud/docker When a folder is opened, I get a 404 with the message "No workspace file found". You seem to have found a solution for Nginx, I am using apache and still searching for a solution. |
Any solution besides disabling rich workspace for snap users? I cannot delete nor modify 403.php/404.php as it is inside the snap folder (Read-only file system). |
Still seeing this issue. Using version 23.03 |
In Nextcloud 24 we replaced the workspace url with parsing properties from the PROPFIND request: #2193. This will therefore be fixed. I'll close this issue. Feel free to reopen it or open another one if you are still seeing the problem from Nextcloud 24 onwards. |
Steps to reproduce
Expected behaviour
Circle progress should disappear after work done
Actual behaviour
Circle progress always spinning
Server configuration
Operating system: Docker Image
Web server: Docker Image
Database: Latest
PHP version: Latest
Nextcloud version: (see Nextcloud admin page)
Nextcloud 18.0.4
Updated from an older Nextcloud/ownCloud or fresh install:
Updated from Nextcloud 18.0.3
Where did you install Nextcloud from:
Docker Image
The text was updated successfully, but these errors were encountered: