sleeper_api 1.10.0 → 1.10.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d1c3b6b24667fed7fcf31880c435c5fce300c4b21dcef51483144385aa2fd13b
-  data.tar.gz: 639011f23fc50f203818f5a035d2c0a15c4e914936d713e10e8b7d357a2d37ef
+  metadata.gz: d50e1154f49ade4542daea9879637a05cacaa3d675718c83630a6800eb399c15
+  data.tar.gz: 3b8814cf8c167653d801dd3a2938f353a4dbe4cb2c219c4baa67cc365e925338
 SHA512:
-  metadata.gz: '091314d14034dfa63a1ebc505e83c457bcbb08e4553e032f08b67917a7d6e6507572aabbc426a00999c770c46d554c48484e64b7a491b615fc97814ef031fd27'
-  data.tar.gz: 03e6caa336e7ff7be7965da2644f2086031a727b0d9eb47824fa1ddf372b74e8ea1f620832304a372169d6ef49b9ef33b2878493ec2da97e690fdabdd072bb4b
+  metadata.gz: bd175da38e1c2e3d460de434744be76c92d6b536ca6a5c7031d90f968a5eb70ebda04424f9a2d888eaa1aefd0283649abcebfd667c7495358233d12a88a86be1
+  data.tar.gz: 502c7f1f64a96eb9e383390e0e42097ca7c8b9c2d24d14aade94e0b99ba8ac342e94b3238ddc7ea24564e75141d6fccc6df9b0c0c313ac29bcb86ee88a1bacfb

data/CHANGELOG.md

@@ -1,3 +1,16 @@
+## [1.10.1] - 2026-10-14
+
+### Fixed
+
+- **A timed-out request is no longer retried behind the gem's back.** Net::HTTP
+  retries an idempotent GET once on its own (`max_retries` defaults to 1), and
+  the gem never switched that off, so every attempt cost two timeouts:
+  `retries = 0` made two requests and `retries = 3` made eight. Against a host
+  that accepts and never answers (measured 2026-10-14), `timeout = 10` gave up
+  after 20.0s and `timeout = 2.5` after 5.0s; now 10.0s and 2.5s. The default
+  configuration's worst case drops from ~243s to ~123s. `retries` is now the
+  only retry.
+
 ## [1.10.0] - 2026-10-14
 
 ### Changed

data/CLAUDE.md

@@ -28,7 +28,7 @@ Four layers, with a deliberate split between HTTP and modeling:
 - **`SleeperApi`** (`lib/sleeper_api.rb`) — module-level config + memoized global `SleeperApi.client`. `Configuration` validates `timeout` (1–60, fractions allowed; 10–60 before 1.10.0) and `retries` (0–5), raising `SleeperApi::Error` outside those bounds.
 - **`Client`** — the only thing that talks HTTP. `include HTTParty` with `base_uri "https://api.sleeper.app"` — **the bare host; the `/v1` lives in each path**. That is deliberate and load-bearing: not every Sleeper endpoint is versioned. `/schedule/{sport}/{season_type}/{season}` is served from the host root, and while `base_uri` carried the version that endpoint was unreachable at any path a caller could pass in. A new endpoint spells out where it lives; do not move the version back into `base_uri` to shorten the paths.
 
-  Every call funnels through the private `make_request`, which handles retry-on-timeout, logging, and converts non-2xx into `SleeperApi::Error` — and, since 1.5.1, **every connection failure too** (`CONNECTION_ERRORS`: `SocketError`, `SystemCallError`, `OpenSSL::SSL::SSLError`, `EOFError`), unretried, as `"Could not reach <path>: …"`. A consumer can rescue `SleeperApi::Error` alone and see every way a call fails. **That error quotes the path**, so the path prefix is part of a public string — v1.2.0 changed it from `"Failed to fetch /user/x: 404"` to `"Failed to fetch /v1/user/x: 404"`. `Client` also owns the 24-hour in-memory player cache.
+  Every call funnels through the private `make_request`, which handles retry-on-timeout (with Net::HTTP's own silent retry switched off, `max_retries: 0`, since 1.10.1 — it doubled every timeout), logging, and converts non-2xx into `SleeperApi::Error` — and, since 1.5.1, **every connection failure too** (`CONNECTION_ERRORS`: `SocketError`, `SystemCallError`, `OpenSSL::SSL::SSLError`, `EOFError`), unretried, as `"Could not reach <path>: …"`. A consumer can rescue `SleeperApi::Error` alone and see every way a call fails. **That error quotes the path**, so the path prefix is part of a public string — v1.2.0 changed it from `"Failed to fetch /user/x: 404"` to `"Failed to fetch /v1/user/x: 404"`. `Client` also owns the 24-hour in-memory player cache.
 
   ⚠️ **Never interpolate a value into a request path; build it with the private `path(*segments)`**, which escapes each segment. An unescaped one raised `URI::InvalidURIError` — not a `SleeperApi::Error`, so it escaped every rescue — or walked out of the endpoint: `../../league/123` as a username requested `/league/123` (G.6, v1.6.0). An unknown user is `200` with a `null` body, which `get_user` turns into `SleeperApi::UserNotFound`.
 

data/lib/sleeper_api/client.rb

@@ -538,7 +538,9 @@ module SleeperApi
     # @raise [SleeperApi::Error] On HTTP errors or timeouts
     def make_request(path, host: nil, not_found: nil)
       named = host ? "#{host}#{path}" : path
-      options = { timeout: @config.timeout }
+      # max_retries: 0, or Net::HTTP retries a timed-out GET once on its own and
+      # every attempt below costs two timeouts.
+      options = { timeout: @config.timeout, max_retries: 0 }
       options[:base_uri] = host if host
 
       @config.logger&.info("Making request to #{host || self.class.base_uri}#{path}")

data/lib/sleeper_api/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SleeperApi
-  VERSION = "1.10.0"
+  VERSION = "1.10.1"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: sleeper_api
 version: !ruby/object:Gem::Version
-  version: 1.10.0
+  version: 1.10.1
 platform: ruby
 authors:
 - Eruity1