nextcloud / notify_push

Update notifications for nextcloud clients
GNU Affero General Public License v3.0
225 stars 40 forks source link

Unrecognized Database URL when using the docker image #404

Closed S0ulf3re closed 5 months ago

S0ulf3re commented 6 months ago

I'm trying to use the notify_push image with my Nextcloud docker instance running on MariaDB. But when I set DATABASE_URL as using MariaDB, I just get an error about an unrecognized database URL. This is my config:

  notify_push:
    image: docker.io/icewind1991/notify_push
    container_name: notify_push
    networks:
      - mariadb
      - redis
      - nextcloud
    environment:
      - NEXTCLOUD_URL="https://nextcloud/"
      - REDIS_URL="redis://redis"
      - DATABASE_URL="mariadb://nextcloud:${NEXTCLOUD_MYSQL_PASSWORD}@db/nextcloud"
    depends_on:
      - db
      - redis
      - nextcloud

I've tried replacing the URI Scheme with mysql as well, but that didn't work either.

joshtrichards commented 5 months ago

DATABASE_URL="mariadb://nextcloud:${NEXTCLOUD_MYSQL_PASSWORD}@db/nextcloud

I suspect this is a Docker interpolation matter or some sort. What happens when specify the entire thing without any variable substitution?

I've tried replacing the URI Scheme with mysql as well, but that didn't work either.

Use mysql for the schema.

S0ulf3re commented 5 months ago

I actually managed to fix this simply by changing this to

notify_push: image: docker.io/icewind1991/notify_push:latest container_name: notify_push networks:

Now I'm trying to figure out how to fix some reverse proxy issues