urllib3 wrapper maintenance: dead kwargs + brittle import-time defaults

[AndresL230]

Summary

Two related urllib3-patch maintenance issues:

1. Dead kwargs

_interceptor.py:108 lists pool_connections and pool_maxsize as kwargs on the urlopen wrapper. Neither is an actual HTTPConnectionPool.urlopen parameter — they're PoolManager.__init__ parameters. The wrapper accepts and silently drops them.

2. Brittle import-time default

_interceptor.py:108 references urllib3.util.Timeout.DEFAULT_TIMEOUT as a default-argument value at function-definition time. If the installed urllib3 (or a future release) doesn't expose that attribute, the SDK fails to import — and the failure happens before any user code runs.

Fix

• Remove the dead pool_connections / pool_maxsize params; rely entirely on **kwargs forwarding.
• Use a sentinel (_DEFAULT = object()) for the timeout default; resolve to the urllib3 default inside the body. Wrap import-time lookups in a try/except so a broken urllib3 install degrades to a no-op rather than crashing the host app.

Files

• recost/_interceptor.py
• tests/test_interceptor.py

Priority

P2 — fragile across urllib3 releases; one renamed attribute could break SDK import for all users on the new urllib3.

[AndresL230]

Audit (Wave B-1) confirms the remaining ask is functionally complete on the code side:

1. Dead kwargs removed. _patched_urlopen signature in recost/_interceptor.py:135 no longer accepts pool_connections / pool_maxsize:

def _patched_urlopen(self: Any, method: str, url: str, body: Any = None,
                    headers: Any = None, retries: Any = None,
                    redirect: bool = True, assert_same_host: bool = True,
                    timeout: Any = _UNSET, **response_kw: Any) -> Any:

2. Import-time default fixed. _UNSET = object() sentinel at _interceptor.py:48. The urllib3.util.Timeout.DEFAULT_TIMEOUT lookup is deferred to function-body time (line 137), so import no longer depends on that attribute existing.

Landed in Wave A (PR #31, commit a47c27f). No further code change required. Closing.