roles/nextcloud: update to new configuration

roles/nextcloud/tasks/main.yml

@@ -6,6 +6,7 @@
     - mariadb-server
     - redis
     - sendmail-bin
+    - libmagickcore-6.q16-6-extra
 
 - name: get and unzip nextcloud
   unarchive:
@@ -33,6 +34,8 @@
     value: "'\\\\OC\\\\Memcache\\\\Redis'"
   - key: "memcache.locking"
     value: "'\\\\OC\\\\Memcache\\\\Redis'"
+  - key: "default_phone_region"
+    value: "'de'"
 
 - name: config acme
   notify: regenerate certificates

roles/nextcloud/templates/nginx

@@ -2,55 +2,18 @@ server {
 	listen [::]:443 http2 ssl;
 	listen 443 http2 ssl;
 	
-        server_name {{ nextcloud_domain }};
+	server_name {{ nextcloud_domain }};
 
 	ssl_certificate /var/lib/dehydrated/certs/{{ nextcloud_domain }}/fullchain.pem;
 	ssl_certificate_key /var/lib/dehydrated/certs/{{ nextcloud_domain }}/privkey.pem;
 
-	# Add headers to serve security related headers
+	# HSTS settings
-	# Before enabling Strict-Transport-Security headers please read into this
-	# topic first.
-	add_header Strict-Transport-Security "max-age=15768000; preload;";
-	#
 	# WARNING: Only add the preload option once you read about
 	# the consequences in https://hstspreload.org/. This option
 	# will add the domain to a hardcoded list that is shipped
 	# in all major browsers and getting removed from this list
 	# could take several months.
-	add_header Referrer-Policy "no-referrer" always;
+	add_header Strict-Transport-Security "max-age=15768000; includeSubDomains; preload;" always;
-	add_header X-Content-Type-Options "nosniff" always;
-	add_header X-Download-Options "noopen" always;
-	add_header X-Frame-Options "SAMEORIGIN" always;
-	add_header X-Permitted-Cross-Domain-Policies "none" always;
-	add_header X-Robots-Tag "none" always;
-	add_header X-XSS-Protection "1; mode=block" always;
-
-	# Remove X-Powered-By, which is an information leak
-	fastcgi_hide_header X-Powered-By;
-
-	root /var/www/nextcloud/;
-
-	location = /robots.txt {
-		allow all;
-		log_not_found off;
-		access_log off;
-	}
-
-	# The following 2 rules are only needed for the user_webfinger app.
-	# Uncomment it if you're planning to use this app.
-	rewrite ^/.well-known/host-meta /public.php?service=host-meta last;
-	rewrite ^/.well-known/host-meta.json /public.php?service=host-meta-json last;
-	
-	# The following rule is only needed for the Social app.
-	# Uncomment it if you're planning to use this app.
-	rewrite ^/.well-known/webfinger /public.php?service=webfinger last;
-
-	location = /.well-known/carddav {
-		return 301 $scheme://$host/remote.php/dav;
-	}
-	location = /.well-known/caldav {
-		return 301 $scheme://$host/remote.php/dav;
-	}
 
 	# set max upload size
 	client_max_body_size 512M;
@@ -64,71 +27,93 @@ server {
 	gzip_proxied expired no-cache no-store private no_last_modified no_etag auth;
 	gzip_types application/atom+xml application/javascript application/json application/ld+json application/manifest+json application/rss+xml application/vnd.geo+json application/vnd.ms-fontobject application/x-font-ttf application/x-web-app-manifest+json application/xhtml+xml application/xml font/opentype image/bmp image/svg+xml image/x-icon text/cache-manifest text/css text/plain text/vcard text/vnd.rim.location.xloc text/vtt text/x-component text/x-cross-domain-policy;
 
-	# Uncomment if your server is build with the ngx_pagespeed module
+	# Pagespeed is not supported by Nextcloud, so if your server is built
-	# This module is currently not supported.
+	# with the `ngx_pagespeed` module, uncomment this line to disable it.
 	#pagespeed off;
 
-	location / {
+	# HTTP response headers borrowed from Nextcloud `.htaccess`
-        	rewrite ^ /index.php;
+	add_header Referrer-Policy                      "no-referrer"   always;
+	add_header X-Content-Type-Options               "nosniff"       always;
+	add_header X-Download-Options                   "noopen"        always;
+	add_header X-Frame-Options                      "SAMEORIGIN"    always;
+	add_header X-Permitted-Cross-Domain-Policies    "none"          always;
+	add_header X-Robots-Tag                         "none"          always;
+	add_header X-XSS-Protection                     "1; mode=block" always;
+
+	# Remove X-Powered-By, which is an information leak
+	fastcgi_hide_header X-Powered-By;
+
+	# Path to the root of your installation
+	root /var/www/nextcloud;
+
+	# Specify how to handle directories -- specifying `/index.php$request_uri`
+	# here as the fallback means that Nginx always exhibits the desired behaviour
+	# when a client requests a path that corresponds to a directory that exists
+	# on the server. In particular, if that directory contains an index.php file,
+	# that file is correctly served; if it doesn't, then the request is passed to
+	# the front-end controller. This consistent behaviour means that we don't need
+	# to specify custom rules for certain paths (e.g. images and other assets,
+	# `/updater`, `/ocm-provider`, `/ocs-provider`), and thus
+	# `try_files $uri $uri/ /index.php$request_uri`
+	# always provides the desired behaviour.
+	index index.php index.html /index.php$request_uri;
+
+	# Rule borrowed from `.htaccess` to handle Microsoft DAV clients
+	location = / {
+		if ( $http_user_agent ~ ^DavClnt ) {
+			return 302 /remote.php/webdav/$is_args$args;
+		}
 	}
 
-	include snippets/tls-acme.conf;
+	location = /robots.txt {
-
+		allow all;
-	location ~ ^\/(?:build|tests|config|lib|3rdparty|templates|data)\/ {
+		log_not_found off;
-		deny all;
+		access_log off;
-	}
-	location ~ ^\/(?:autotest|occ|issue|indie|db_|console) {
-		deny all;
 	}
 
-	location ~ ^\/(?:index|remote|public|cron|core\/ajax\/update|status|ocs\/v[12]|updater\/.+|oc[ms]-provider\/.+)\.php(?:$|\/) {
+	location ^~ /.well-known {
-		fastcgi_split_path_info ^(.+?\.php)(\/.*|)$;
+		# The following 6 rules are borrowed from `.htaccess`
+
+		location = /.well-known/carddav     { return 301 /remote.php/dav/; }
+		location = /.well-known/caldav      { return 301 /remote.php/dav/; }
+		include snippets/tls-acme.conf;
+		# Anything else is dynamically handled by Nextcloud
+		location ^~ /.well-known            { return 301 /index.php$uri; }
+
+		try_files $uri $uri/ =404;
+	}
+
+	# Rules borrowed from `.htaccess` to hide certain paths from clients
+	location ~ ^/(?:build|tests|config|lib|3rdparty|templates|data)(?:$|/)  { return 404; }
+	location ~ ^/(?:\.|autotest|occ|issue|indie|db_|console)              { return 404; }
+
+	# Ensure this block, which passes PHP files to the PHP process, is above the blocks
+	# which handle static assets (as seen below). If this block is not declared first,
+	# then Nginx will encounter an infinite rewriting loop when it prepends `/index.php`
+	# to the URI, resulting in a HTTP 500 error response.
+	location ~ \.php(?:$|/) {
+		fastcgi_split_path_info ^(.+?\.php)(/.*)$;
 		set $path_info $fastcgi_path_info;
+
 		try_files $fastcgi_script_name =404;
+
 		include fastcgi_params;
 		fastcgi_param SCRIPT_FILENAME $document_root$fastcgi_script_name;
 		fastcgi_param PATH_INFO $path_info;
 		fastcgi_param HTTPS on;
-		# Avoid sending the security headers twice
+
-		fastcgi_param modHeadersAvailable true;
+		fastcgi_param modHeadersAvailable true;         # Avoid sending the security headers twice
-		# Enable pretty urls
+		fastcgi_param front_controller_active true;     # Enable pretty urls
-		fastcgi_param front_controller_active true;
 		fastcgi_pass unix:{{ nextcloud_php_fpm }};
+
 		fastcgi_intercept_errors on;
 		fastcgi_request_buffering off;
 	}
 
-	location ~ ^\/(?:updater|oc[ms]-provider)(?:$|\/) {
+	location ~ \.(?:css|js|svg|gif)$ {
-		try_files $uri/ =404;
-		index index.php;
-	}
-
-	# Adding the cache control header for js and css files
-	# Make sure it is BELOW the PHP block
-	location ~ \.(?:css|js|woff2?|svg|gif|map)$ {
 		try_files $uri /index.php$request_uri;
-		add_header Cache-Control "public, max-age=15778463";
+		expires 6M;         # Cache-Control policy borrowed from `.htaccess`
-		# Add headers to serve security related headers (It is intended to
+		access_log off;     # Optional: Don't log access to assets
-		# have those duplicated to the ones above)
-		# Before enabling Strict-Transport-Security headers please read into
-		# this topic first.
-		add_header Strict-Transport-Security "max-age=15768000; preload;";
-		#
-		# WARNING: Only add the preload option once you read about
-		# the consequences in https://hstspreload.org/. This option
-		# will add the domain to a hardcoded list that is shipped
-		# in all major browsers and getting removed from this list
-		# could take several months.
-		# Optional: Don't log access to assets
-		add_header Referrer-Policy "no-referrer" always;
-		add_header X-Content-Type-Options "nosniff" always;
-		add_header X-Download-Options "noopen" always;
-		add_header X-Frame-Options "SAMEORIGIN" always;
-		add_header X-Permitted-Cross-Domain-Policies "none" always;
-		add_header X-Robots-Tag "none" always;
-		add_header X-XSS-Protection "1; mode=block" always;
-
-		access_log off;
 	}
 
 	location ~ \.(?:png|html|ttf|ico|jpg|jpeg)$ {
@@ -136,4 +121,14 @@ server {
 		# Optional: Don't log access to other assets
 		access_log off;
 	}
+
+	location ~ \.woff2?$ {
+		try_files $uri /index.php$request_uri;
+		expires 7d;         # Cache-Control policy borrowed from `.htaccess`
+		access_log off;     # Optional: Don't log access to assets
+	}
+
+	location / {
+		try_files $uri $uri/ /index.php$request_uri;
+	}
 }