Fix notify_push sometimes using wrong URL

bin/ncp-update-nc

@@ -199,12 +199,14 @@ else
     cat > /etc/systemd/system/notify_push.service <<EOF
 [Unit]
 Description = Push daemon for Nextcloud clients
-After = mysql.service
+After=mysql.service
+After=redis.service
+Requires=redis.service
 
 [Service]
-Environment = PORT=7867
+Environment=PORT=7867
 Environment=NEXTCLOUD_URL=https://localhost
-ExecStart = /var/www/nextcloud/apps/notify_push/bin/"${arch}"/notify_push --allow-self-signed /var/www/nextcloud/config/config.php
+ExecStart=/var/www/nextcloud/apps/notify_push/bin/"${arch}"/notify_push --allow-self-signed /var/www/nextcloud/config/config.php
 User=www-data
 
 [Install]

changelog.md

@@ -1,4 +1,7 @@
+[v1.48.1](https://github.com/nextcloud/nextcloudpi/tree/v1.48.2) (2022-07-20) Fix notify_push using invalid domain; Fix Nextcloud v23.0.6 not being installed by default 
+
 [v1.48.1](https://github.com/nextcloud/nextcloudpi/tree/v1.48.1) (2022-06-12) Fix docker release missing latest tags
+
 [v1.48.0](https://github.com/nextcloud/nextcloudpi/tree/v1.48.0) (2022-06-12) Update Nextcloud to version 23.0.6
 
 [v1.47.2](https://github.com/nextcloud/nextcloudpi/commit/b66ce17) (2022-05-05) docker-build.yml: Add CI/CD integration tests for docker images

updates/1.48.2.sh

@@ -0,0 +1,30 @@
+#!/bin/bash
+
+set -e
+
+source /usr/local/etc/library.sh
+
+
+is_docker || {
+  arch="$(uname -m)"
+  [[ "${arch}" =~ "armv7" ]] && arch="armv7"
+  cat > /etc/systemd/system/notify_push.service <<EOF
+[Unit]
+Description = Push daemon for Nextcloud clients
+After=mysql.service
+After=redis.service
+Requires=redis.service
+
+[Service]
+Environment=PORT=7867
+Environment=NEXTCLOUD_URL=https://localhost
+ExecStart=/var/www/nextcloud/apps/notify_push/bin/"${arch}"/notify_push --allow-self-signed /var/www/nextcloud/config/config.php
+User=www-data
+
+[Install]
+WantedBy = multi-user.target
+EOF
+  start_notify_push
+}
+
+exit 0