-
-
Notifications
You must be signed in to change notification settings - Fork 499
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
nextcloud memories, Missing Line in nginx-nextcloud.conf #7192
Comments
Maps Work again also (App Maps opened, but No layer was displayed) |
Hmm, this does not seem to be supported widely among browsers, e.g. Safari and certain mobile browsers do not support it at all:
Isn't it possible to do all this with CSP? Nextcloud has an API for this, which can be used by apps, if I am not mistaken. This does not seem suitable to add for all Nextcloud instances, also it should be probably added for the very specific URLs used by this app only, instead of disabling CORS entirely for the whole Nextcloud instance. This all can be done much better with CSP from the backend/app code, instead of globally via webserver configuration. I suggest you talk to its developer/maintainer about this. Also, why did you add it twice? |
I added it twice in two locations in this config file: Yesterday i updated to nextcould 30.0.0 (via beta channel) in dietpi 9.7.1
I did not know about CSP. I did not touch any nextcloud files (but the nextcloud.conf). CSP editor says " no custom policy set". Isn´t the dietpi-nextcloud.conf only for nextcloud but not for global webserver config? |
I just did a test installation of Nextcloud with Nginx, installed the maps app, and it works OOTB here, if you mean map tiles: Can you check in your browser console/developer tools, which rule exactly denies loading of this resources? Probably your Nginx globally overrides CSP or so, breaking the apps. |
I tried it with two browsers and i thinks these are the errors for not loading tiles: firefox131.0: multiprocess: errors: edge 129.0.2792.65: Copilot says Copilot Gesendet von Copilot:
Which nginx config file could be correct? in sites-enabled/default and sites-available/default there are a few lines by Certbot This is my /etc/nginx/sites-dietpi/ config-file: |
Hmm, that header should not be set by default. Can you check this: grep -ri 'Cross-Origin-Resource-Policy' /etc/nginx |
|
Hmm, and when you run this: curl -Ik https://localhost
curl -Ik https://localhost/nextcloud/ |
curl -Ik https://localhost
curl -Ik https://localhost/nextcloud/
|
i restarted service nginx, memories do not load again, no previews for all pictures, but grey boxes are displayed. When clicking the box on top (last year), the image is displayed. When clicking boxes on bottom (timeline), no image is displayed. |
Hmm, no |
There is pihole on the same device. It is configured in the Fritzbox as DNS. |
When opening the app in browser, can you check the response headers like this Hit F12 to open developer tools, switch to Network tab, reload page with F5. At the top of the request list, select As you can see in my case, there is no |
I searched for Cross-Origin-Embedder-Policy: require-corp and came to failure was described before So i disabled and uninstalled Nextcloud office 8.5.1 and both memories and maps work as expected. |
Oh great find, so it is the Nextcloud Office app which sets this globally. That is of course a problem. I'll check back with those guys, as this seems to be wrong to me. |
Creating a bug report/issue
Required Information
Additional Information (if applicable)
Steps to reproduce
Install nginx, nextcloud,nc App memories, lets-encrypt
Expected behaviour
Memories should Display thumbs and images
Actual behaviour
Thumbs are generated, but are Gray and images are Not displayed
Extra details
Solution is to add Line in /etc/nginx/sites-available/dietpi-nextcloud.conf
I added It twice, IT works AS IT should now
pulsejet/memories#1236 (comment)
Regards, droogi
The text was updated successfully, but these errors were encountered: