YunoHost-Apps / nextcloud_ynh

Nextcloud package for YunoHost
https://nextcloud.com/
GNU Affero General Public License v3.0
144 stars 62 forks source link

nginx.conf for arm needs small adjustment #657

Closed majamee closed 5 months ago

majamee commented 5 months ago

I'm using YunoHost 11.2.9.1 (stable) and NextCloud 28.0.1~ynh2 on ARM64.

Since a short while a there is an ARM64 version of Nextclouds internal Collabora CODE server available: https://apps.nextcloud.com/apps/richdocumentscode_arm64

This turns out to work well as soon as you modify the nginx.conf as follows: https://github.com/YunoHost-Apps/nextcloud_ynh/blob/master/conf/nginx.conf

Change line (currently 105) from: rewrite ^__PATH__/(?!index|remote|public|cron|core\/ajax\/update|status|ocs\/v[12]|updater\/.+|ocs-provider\/.+|.+\/richdocumentscode\/proxy) __PATH__/index.php$request_uri;

To be (change required for ARM64 version only): rewrite ^__PATH__/(?!index|remote|public|cron|core\/ajax\/update|status|ocs\/v[12]|updater\/.+|ocs-provider\/.+|.+\/richdocumentscode_arm64\/proxy) __PATH__/index.php$request_uri;

As soon as you have done so everything is working as expected. Would be great, if that config adjustment could be included for ARM64 based systems. Thanks a lot for your effort, I hope I could be helpful :)

majamee commented 5 months ago

I guess, the easiest way would be to expand the rewrite line to include both redirects, e.g.:

rewrite ^/nextcloud/(?!index|remote|public|cron|core\/ajax\/update|status|ocs\/v[12]|updater\/.+|ocs-provider\/.+|.+\/richdocumentscode\/proxy|.+\/richdocumentscode_arm64\/proxy) /nextcloud/index.php$request_uri;

This should work for both architectures (x86_64 and ARM64) :)