diff --git a/redis/connection.py b/redis/connection.py index f745ecc1d5..728c221257 100644 --- a/redis/connection.py +++ b/redis/connection.py @@ -909,9 +909,9 @@ class UnixDomainSocketConnection(AbstractConnection): "Manages UDS communication to and from a Redis server" def __init__(self, path="", socket_timeout=None, **kwargs): + super().__init__(**kwargs) self.path = path self.socket_timeout = socket_timeout - super().__init__(**kwargs) def repr_pieces(self): pieces = [("path", self.path), ("db", self.db)] diff --git a/tests/test_connect.py b/tests/test_connect.py index 71986dd8f3..d7ca04b651 100644 --- a/tests/test_connect.py +++ b/tests/test_connect.py @@ -98,6 +98,23 @@ def test_tcp_ssl_tls12_custom_ciphers(tcp_address, ssl_ciphers): _assert_connect(conn, tcp_address, certfile=certfile, keyfile=keyfile) +""" +Addresses bug CAE-333 which uncovered that the init method of the base +class did override the initialization of the socket_timeout parameter. +""" + + +def test_unix_socket_with_timeout(): + conn = UnixDomainSocketConnection(socket_timeout=1000) + + # Check if the base class defaults were taken over. + assert conn.db == 0 + + # Verify if the timeout and the path is set correctly. + assert conn.socket_timeout == 1000 + assert conn.path == "" + + @pytest.mark.ssl @pytest.mark.skipif(not ssl.HAS_TLSv1_3, reason="requires TLSv1.3") def test_tcp_ssl_version_mismatch(tcp_address):