fix: bump redis floor to 5.0.1, disconnect pool on failed connect(), doc/type nits (redis-5/6/7/8); v0.1.4

redis-5: redis>=5 admitted 5.0.0, which lacks async aclose() and crashes on
teardown; floor bumped to >=5.0.1 (confirmed 5.0.1 is where aclose() lands).
redis-6: connect()/__aenter__ now disconnects the pool before re-raising on a
failed ping, so a discarded RedisDB from a failed `async with` doesn't hold a
live pool. redis-7: docstring/README conflated repo name (redis) with
distribution name (redis_store). redis-8: pool_timeout hinted Optional[float]
to match that None is accepted (wait forever). Also compresses the module and
__init__ docstrings (no behavior change), keeping the SSLConnection-mapping
and pool-disconnect-on-failure notes.

Signed-off-by: disqualifier <dev@disqualifier.me>

README.md

@@ -4,7 +4,7 @@ Async Redis wrapper over redis-py's asyncio client — a small, config-free, **f
 key/value + hash + ttl surface with a raw escape hatch for everything else. First of the
 datastore trio (`redis` / `psql` / `mysql`), a sibling of the `mongo` lib.
 
-> **Import name ≠ repo name.** The repo/distribution is **`redis`**, but you import
+> **Import name ≠ repo name.** The repo is **`redis`**, the distribution is
 > **`redis_store`** — the driver package owns the `redis` import namespace, so the lib
 > can't also be `redis`. Install resolves `redis.git`; code does
 > `from redis_store import RedisDB`.
@@ -14,19 +14,19 @@ datastore trio (`redis` / `psql` / `mysql`), a sibling of the `mongo` lib.
 `requirements.txt`:
 
 ```
-redis_store @ git+ssh://git@git.rethinkstudios.io/rethink-public/redis.git@v0.1.2
+redis_store @ git+ssh://git@git.rethinkstudios.io/rethink-public/redis.git@v0.1.4
 ```
 
 Direct:
 
 ```bash
-pip install "redis_store @ git+ssh://git@git.rethinkstudios.io/rethink-public/redis.git@v0.1.2"
+pip install "redis_store @ git+ssh://git@git.rethinkstudios.io/rethink-public/redis.git@v0.1.4"
 ```
 
-Pulls `redis>=5` (redis-py, which ships the asyncio client — **not** the dead standalone
-`aioredis`).
+Pulls `redis>=5.0.1` (redis-py, which ships the asyncio client — **not** the dead standalone
+`aioredis`; `5.0.1` is the floor because `5.0.0` lacks the async client's `aclose()`).
 
-Drop the `@v0.1.2` suffix from the line above to install the latest unpinned.
+Drop the `@v0.1.4` suffix from the line above to install the latest unpinned.
 
 ## Usage
 
@@ -55,6 +55,8 @@ async with RedisDB(host="localhost") as kv:
 ```
 
 One client/pool per process — build it once, attach it to your app (`app.kv = ...`), share it.
+A failed `connect()`/`__aenter__` ping disconnects the pool before re-raising, so a
+discarded `RedisDB` from a failed `async with` doesn't hold a live pool.
 
 ## Type contract
 
@@ -63,8 +65,14 @@ One client/pool per process — build it once, attach it to your app (`app.kv =
 Counters and counts (`incr`/`decr`/`exists`/`ttl`) always return `int` regardless.
 
 Pool sizing/timeout are `max_connections=` and `pool_timeout=` (how long a caller waits for
-a free connection when the pool is saturated). Pass the driver's own `timeout=` in
-`pool_kwargs` and construction raises a clear `ValueError` — use `pool_timeout=`.
+a free connection when the pool is saturated; `Optional[float]`, `None` waits forever).
+Pass the driver's own `timeout=` in `pool_kwargs` and construction raises a clear
+`ValueError` — use `pool_timeout=`.
+
+`ssl=True` maps to `connection_class=redis.asyncio.SSLConnection`, so TLS works via the
+documented `pool_kwargs` path (the default connection class has no `ssl` parameter and
+would otherwise defer a `TypeError` to `connect()`). Pass ssl-prefixed kwargs alongside
+it (`ssl_ca_certs`, `ssl_certfile`, ...) to configure the handshake.
 
 ## Error contract — fail loud
 

pyproject.toml

@@ -4,11 +4,11 @@ build-backend = "hatchling.build"
 
 [project]
 name = "redis_store"
-version = "0.1.2"
+version = "0.1.4"
 description = "async redis wrapper over redis-py asyncio with a raw escape hatch, fail-loud and config-free"
 requires-python = ">=3.10"
 dependencies = [
-    "redis>=5",
+    "redis>=5.0.1",
 ]
 
 [tool.hatch.build.targets.wheel]

src/redis_store/__init__.py

@@ -1,3 +1,5 @@
 from .redis_store import RedisDB
 
-__all__ = ["RedisDB"]
+__version__ = "0.1.4"
+
+__all__ = ["RedisDB", "__version__"]

src/redis_store/redis_store.py

@@ -12,28 +12,27 @@ context manager:
     async with RedisDB(host="localhost") as kv:
         await kv.incr("hits")
 
-lifecycle:
-    construction is sync and opens no socket (the pool connects lazily). connect()
-    issues a ping() so a bad host/port/auth fails loud immediately rather than on the
-    first real op, and returns self for the one-liner above. close() tears the client
-    and pool down (await client.aclose() + pool.disconnect()).
+lifecycle: construction is sync, opens no socket (pool connects lazily). connect()
+    pings so bad host/port/auth fails loud immediately, returns self. close() tears
+    down client + pool (client.aclose() + pool.disconnect()). a failed connect()/
+    __aenter__ ping also disconnects the pool before re-raising, so a discarded
+    RedisDB from a failed `async with` doesn't hold a live pool.
 
-type contract:
-    decode_responses=True by default, so keys and string values come back as str (and
-    None for an absent key). pass decode_responses=False at construction if you need raw
-    bytes. counters (incr/decr/exists/ttl) always return int regardless of this flag.
+type contract: decode_responses=True by default -> str values, None for absent key.
+    decode_responses=False for raw bytes. counters (incr/decr/exists/ttl) always int.
 
-errors (FAIL LOUD — unlike the mongo lib's swallow-and-default):
-    every wrapped method catches the driver's RedisError, logs it via
-    getLogger(__name__), and re-raises. a None/[]/{} return is only ever a real result
-    (absent key, empty hash) — never a swallowed failure. for anything not wrapped
-    (pipelines, pub/sub, scan, Lua, ...) use the raw `.client` escape hatch, which
-    exposes the full redis.asyncio.Redis surface and raises untouched.
+errors (FAIL LOUD — unlike mongo's swallow-and-default): every wrapped method catches
+    RedisError, logs via getLogger(__name__), re-raises. None/[]/{} is only ever a
+    real result, never a swallowed failure. unwrapped ops (pipelines, pub/sub, scan,
+    Lua, ...) go through the raw `.client` escape hatch: full driver surface, raises.
 
 notes:
-    - import is `redis_store`; the repo/distribution is `redis` (the driver owns the
-      `redis` import name, so the package can't also be `redis`)
-    - pub/sub and pipeline helpers are intentionally not wrapped yet — use `.client`
+    - import is `redis_store`; repo is `redis`, distribution is `redis_store` (the
+      driver owns the `redis` import name, so the package can't also be `redis`)
+    - pub/sub and pipeline helpers intentionally not wrapped yet — use `.client`
+    - ssl=True maps to connection_class=redis.asyncio.SSLConnection so TLS works via
+      the documented pool_kwargs path; pass ssl-prefixed kwargs alongside it
+      (ssl_ca_certs, ssl_certfile, ...) to configure the handshake
 """
 
 import logging
@@ -56,31 +55,40 @@ class RedisDB:
         password: Optional[str] = None,
         *,
         max_connections: int = 10,
-        pool_timeout: float = 30.0,
+        pool_timeout: Optional[float] = 30.0,
         decode_responses: bool = True,
         **pool_kwargs,
     ):
         """build the (not-yet-connected) pool + client; no I/O happens here
 
-        host/port/db/password/max_connections are injected by the caller. extra
+        host/port/db/password/max_connections injected by the caller. extra
         pool_kwargs pass through to the connection pool (socket_timeout,
-        socket_connect_timeout, ssl, etc). decode_responses=True returns str; set it
-        False for raw bytes.
+        socket_connect_timeout, ssl, etc). decode_responses=True returns str, False
+        raw bytes.
 
-        a BlockingConnectionPool is used so that, under concurrency exceeding
-        max_connections, callers WAIT up to pool_timeout seconds for a free connection
-        (matching motor/mongo's blocking pool) rather than the plain pool's behavior of
-        raising MaxConnectionsError immediately. pool_timeout=None waits forever.
+        a BlockingConnectionPool is used so callers WAIT up to pool_timeout seconds
+        for a free connection under concurrency exceeding max_connections (matching
+        motor/mongo's blocking pool), instead of raising MaxConnectionsError
+        immediately. pool_timeout=None waits forever.
 
-        `timeout` is the pool's own kwarg (set from pool_timeout here), so passing it in
-        pool_kwargs would collide — reject it with a clear pointer rather than let the
-        driver raise a cryptic "multiple values for 'timeout'".
+        `timeout` is the pool's own kwarg (filled from pool_timeout here); passing it
+        in pool_kwargs would collide, so it's rejected with a clear ValueError instead
+        of a cryptic driver "multiple values for 'timeout'".
+
+        `ssl=True` maps to `connection_class=redis.asyncio.SSLConnection` — the
+        default async Connection class has no `ssl` parameter, so passing it straight
+        through pool_kwargs would defer the failure to connect() as a raw TypeError.
+        pass ssl-prefixed kwargs (ssl_ca_certs, ssl_certfile, ...) alongside ssl=True
+        to configure TLS; an explicit connection_class in pool_kwargs is left
+        untouched.
         """
         if "timeout" in pool_kwargs:
             raise ValueError(
                 "redis_store: pass the pool wait timeout as pool_timeout=, not timeout= "
                 "(timeout is filled from pool_timeout)"
             )
+        if pool_kwargs.pop("ssl", False):
+            pool_kwargs.setdefault("connection_class", redis.SSLConnection)
         self._pool = redis.BlockingConnectionPool(
             host=host,
             port=port,
@@ -103,6 +111,7 @@ class RedisDB:
             await self._client.ping()
         except RedisError:
             log.exception("redis.connect() ping failed")
+            await self._pool.disconnect()
             raise
         return self