Skip to content

Commit

Permalink
Merge pull request #3398 from nextcloud/enh/3389/improve-notify-push
Browse files Browse the repository at this point in the history
add warning to notify-push if binary was not found
  • Loading branch information
szaimen authored Sep 27, 2023
2 parents 4bc426d + 6790114 commit 142c605
Showing 1 changed file with 15 additions and 0 deletions.
15 changes: 15 additions & 0 deletions Containers/notify-push/start.sh
Original file line number Diff line number Diff line change
Expand Up @@ -27,6 +27,21 @@ elif [ "$CPU_ARCH" != "x86_64" ]; then
export CPU_ARCH="aarch64"
fi

# Add warning
if ! [ -f /nextcloud/custom_apps/notify_push/bin/"$CPU_ARCH"/notify_push ]; then
echo "The notify_push binary was not found."
echo "Most likely is DNS resolution not working correctly."
echo "You can try to fix this by configuring a DNS server globally in dockers daemon.json."
echo "See https://dockerlabs.collabnix.com/intermediate/networking/Configuring_DNS.html"
echo "Afterwards a restart of docker should automatically resolve this."
echo "Additionally, make sure to disable VPN software that might be running on your server"
echo "Also check your firewall if it blocks connections to github"
echo "If it should still not work afterwards, feel free to create a new thread at https://github.com/nextcloud/all-in-one/discussions/new?category=questions and post the Nextcloud container logs there."
echo ""
echo ""
exit 1
fi

# Set sensitive values as env
export DATABASE_URL="postgres://$POSTGRES_USER:$POSTGRES_PASSWORD@$POSTGRES_HOST/$POSTGRES_DB"
export REDIS_URL="redis://:$REDIS_HOST_PASSWORD@$REDIS_HOST"
Expand Down

0 comments on commit 142c605

Please sign in to comment.