shohamy7 commented on PR #36562:
URL: https://github.com/apache/airflow/pull/36562#issuecomment-1875913688

   @hussein-awala 
   Currently, in the redis provider we are requiring at least `redis>=4.5.2` 
which support Redis versions 5.0 to 7.0.
   From the [redis docs](https://redis.io/docs/management/security/acl/), it 
looks like there is backward compatibility (when we are not specifying the 
username, the `default` user will be used).
   From the redis client perspective, the `username` field was always there - 
we just did not set it (so it was `None`).
   If you won't set the login field in the connection form - the username will 
be `None` and the `RedisHook` will act the same.
   When using old Redis versions (e.g. 5.0) which not support username 
authentication - it should be handle by the redis client, as they are saying 
that it is supported :)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to