fix:removing use of self.protocol to support redis-py<5

Author: cunla

docs/about/changelog.md

@@ -7,6 +7,12 @@ tags:
 toc_depth: 2
 ---
 
+## v2.31.4 -
+
+### 🐛 Bug Fixes
+
+- fix:removing use of self.protocol to support redis-py < 5 #419
+
 ## v2.31.3 -
 
 ### 🐛 Bug Fixes

fakeredis/_connection.py

@@ -24,6 +24,7 @@ def connect(self) -> None:
     def _connect(self) -> FakeSocket:
         if not self._server.connected:
             raise redis.ConnectionError(msgs.CONNECTION_ERROR_MSG)
+        protocol = getattr(self, "protocol", 2)
         return FakeSocket(
             self._server,
             client_class=self._client_class,
@@ -56,7 +57,7 @@ def _connect(self) -> FakeSocket:
                 cmd="auth",
                 user="default",
                 redir=-1,
-                resp=self.protocol,
+                resp=protocol,
             ),
         )
 

pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "fakeredis"
-version = "2.31.3"
+version = "2.31.4"
 description = "Python implementation of redis API, can be used for testing purposes."
 authors = [
     { name = "Daniel Moran", email = "daniel@moransoftware.ca" },

test/test_internals/test_init_args.py

@@ -67,6 +67,10 @@ def test_from_url(self):
         db.set("foo", "bar")
         assert db.get("foo") == b"bar"
 
+    def test_from_url_ping(self):
+        client = fakeredis.FakeRedis.from_url("redis://localhost:6390/0")
+        client.ping()
+
     def test_from_url_user(self):
         username = "fakeredis-user"
         db = fakeredis.FakeStrictRedis(host="localhost", port=6390, db=0)