fix: match retry_methods case-insensitively

Author: lesnik512

docs/resilience.md

@@ -27,7 +27,7 @@ from httpware.middleware.resilience import AsyncRetry
 | `base_delay` | `0.1` (s) | Floor for the full-jitter exponential backoff. Must be `>= 0`; a negative or NaN value raises `ValueError`. |
 | `max_delay` | `5.0` (s) | Ceiling for backoff and the longest `Retry-After` honored. Must be finite and `>= 0`; a negative, NaN or infinite value raises `ValueError`. |
 | `retry_status_codes` | `frozenset({408, 429, 502, 503, 504})` | Status codes considered retryable. |
-| `retry_methods` | `frozenset({"GET", "HEAD", "OPTIONS", "PUT", "DELETE"})` | Idempotent methods only. To retry POST, pass a frozenset that includes `"POST"`. |
+| `retry_methods` | `frozenset({"GET", "HEAD", "OPTIONS", "PUT", "DELETE"})` | Idempotent methods only. To retry POST, pass a frozenset that includes `"POST"`. Matching ignores case, so `"post"` works too. |
 | `respect_retry_after` | `True` | When a retryable response has a `Retry-After` header, wait that long instead of the jittered backoff. If the value is over `max_delay`, `AsyncRetry` gives up and re-raises the `StatusError` with a PEP 678 note: `httpware: Retry-After (Ns) exceeded max_delay (Ms); giving up`. To avoid this, set `respect_retry_after=False` or raise `max_delay`. |
 | `budget` | `RetryBudget()` | The token bucket. Pass one shared `RetryBudget` to several clients to give them a joint budget. |
 

src/httpware/middleware/resilience/retry.py

@@ -122,7 +122,7 @@ def __init__(  # noqa: PLR0913 — retry policy has many orthogonal knobs; a dat
         self.base_delay = base_delay
         self.max_delay = max_delay
         self.retry_status_codes = retry_status_codes
-        self.retry_methods = retry_methods
+        self.retry_methods = frozenset(method.upper() for method in retry_methods)
         self.respect_retry_after = respect_retry_after
         self.budget = budget if budget is not None else RetryBudget()
 
@@ -139,7 +139,7 @@ def decide(  # noqa: C901 — complexity budget: classification + streaming-body
         exception. `exc` is the currently-handled exception (one of
         _RETRYABLE_EXCEPTIONS); see the class docstring for chaining semantics.
         """
-        method_eligible = request.method.upper() in self.retry_methods
+        method_eligible = request.method in self.retry_methods
         if isinstance(exc, StatusError):
             retryable_status = exc.response.status_code in self.retry_status_codes
             if not method_eligible or not retryable_status:

tests/test_retry.py

@@ -128,6 +128,18 @@ async def test_retries_post_when_method_explicitly_included() -> None:
     assert handler.calls == 2  # noqa: PLR2004 — "2" is intentional literal in test assertion
 
 
+async def test_retry_methods_match_case_insensitively() -> None:
+    sleeper = _SleepRecorder()
+    handler = _ResponseSequence([HTTPStatus.SERVICE_UNAVAILABLE, HTTPStatus.OK])
+    client = _client(
+        handler,
+        retry=AsyncRetry(_sleep=sleeper, retry_methods=frozenset({"post"}), base_delay=0.01, max_delay=0.02),
+    )
+    response = await client.post("https://example.test/x", json={"x": 1})
+    assert response.status_code == HTTPStatus.OK
+    assert handler.calls == 2  # noqa: PLR2004 — initial attempt + 1 retry
+
+
 async def test_max_attempts_one_means_no_retries() -> None:
     sleeper = _SleepRecorder()
     handler = _ResponseSequence([HTTPStatus.SERVICE_UNAVAILABLE])

tests/test_retry_sync.py

@@ -457,6 +457,18 @@ def test_retries_post_when_method_explicitly_included() -> None:
     assert len(sleeper.calls) == 1
 
 
+def test_retry_methods_match_case_insensitively() -> None:
+    sleeper = _SleepRecorder()
+    handler = _ResponseSequence([HTTPStatus.SERVICE_UNAVAILABLE, HTTPStatus.OK])
+    client = _client(
+        handler,
+        retry=Retry(_sleep=sleeper, retry_methods=frozenset({"post"}), base_delay=0.01, max_delay=0.02),
+    )
+    response = client.post("https://example.test/x", json={"k": "v"})
+    assert response.status_code == HTTPStatus.OK
+    assert handler.calls == 2  # noqa: PLR2004 — initial attempt + 1 retry
+
+
 def test_default_budget_is_fresh_per_instance() -> None:
     r1 = Retry()
     r2 = Retry()