notify_push icon indicating copy to clipboard operation
notify_push copied to clipboard

Redis TLS/User+Pass not working

Open solracsf opened this issue 2 years ago • 3 comments

This works on server:

'redis' =>
  array (
    'host' => 'tls://redis-8d5a784c-o8161d3b7.database',
    'port' => 20185,
    'user' => 'bast',
    'password' => 'AVNS_4zX7t5j',
    'dbindex' => 38,
    'timeout' => 10.0,
    'read_timeout' => 10.0,
    'retry_interval' => 500,
  ),

but notify_push can't connect to it:

Failed to setup redis subscription: Error while running redis query
WARN [notify_push] src/lib.rs:418: Redis server disconnected, reconnecting in 1s

solracsf avatar Jun 05 '22 22:06 solracsf

I haven't tried with TLS, but I can confirm that the simple fact of having a Redis password does not works. By giving a quick look and search in the source code, it seems like that only the Redis URL is parsed and used, not the password value (however I could be wrong on this). Maybe it's not a bug, more like a feature not developed yet.

EDIT : My bad, my redis ACL were allowing NC to use it, but not the notify_push binary, even if it uses the same user. Now it works as expected, but still without TLS.

ThibautPlg avatar Dec 27 '22 13:12 ThibautPlg

The latest release includes improved error messages for redis errors, please retry with the latest release

icewind1991 avatar Feb 10 '23 11:02 icewind1991

hi, i encountered this issue too, the daemon service log is looping like this:

ERROR [notify_push] src/lib.rs:410: Failed to setup redis subscription: failed to lookup address information: Name does not resolve
WARN [notify_push] src/lib.rs:412: Redis server disconnected, reconnecting in 1s

output from occ notify_push setup :

✓ redis is configured
🗴 push server is not receiving redis messages (received 340330037, got 0)

i'm using client push 0.6.5 nextcloud 27.1.3 debian 12 redis 7 (managed digital ocean db, so forced to use tls)

nurradityam avatar Nov 12 '23 17:11 nurradityam