gabe565 / charts

☸️ A collection of Helm charts, based on the bjw-s common library chart.
https://charts.gabe565.com
Apache License 2.0
109 stars 20 forks source link

paperless-ngx Redis Connection Fails #551

Open RikuXan opened 9 months ago

RikuXan commented 9 months ago

Chart Name

paperless-ngx

Chart Version

0.15.1

Bug Description

My paperless-ngx cannot connect to Redis anymore, printing out an authentication error. Using the same Redis connection URL as in the paperless-ngx environment variable via redis-cli inside the Redis container gives me the same error. To connect successfully I have to either explicitly include the default username inside the connection string or alternatively remove the colon before the password.

values.yaml

No response

Relevant log output

No response

gabe565 commented 9 months ago

@RikuXan I suspect this is related to the recent Redis Helm chart update. I'm working on a fix!

gabe565 commented 9 months ago

@RikuXan It's interesting because their Redis env documentation still shows the syntax including the :, so I'm hesitant to update the default value. I'm able to replicate, though. Hmm

gabe565 commented 9 months ago

@RikuXan Oh whoops...I'm not actually able to replicate. It was a network policy issue in my setup. My pod seems to be healthy, and the env got set to PAPERLESS_REDIS=redis://:KZ8fXPJJSC@paperless-ngx-redis-master. Do you have any redis configuration set other than the default?

RikuXan commented 9 months ago

@gabe565 No, my Redis configuration is completely default. The only customizations I'm currently using in the overall chart are:

My Redis connection string looks the same as yours, but using it via redis-cli (redis-cli -u <connection-string>) inside the Redis container gives me the aforementioned error. The Redis container's image version is 7.2.4-debian-11-r0.

If it's not an issue for you, a solution might be to include a variable that is empty by default, but can be customized for the username. This way people who experience the issue could customize their Redis connection string, as well as creating compatibility with externally-hosted Redis instances with ACLs enabled (where the username can then actually be relevant as far as I understand it).