barkibu-kb 0.31.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 35bd731195eab428b21d95b6a7e8c84a78017d67089a90e99175eaaf3bfe9df8
-  data.tar.gz: 7ee5422696a8e15488c30d93a1c26d853b151201cb3b85225a7f3f48c8d2baff
+  metadata.gz: 5eb3a704f422c511e2e30d584cd6a80a7d91a090cfa5452dd197e65496b52eee
+  data.tar.gz: 05d766aeca3b869601193bd1a6310ced1c07f96d6f24cd5424c290a33b1edaf9
 SHA512:
-  metadata.gz: 239cf144f088a79e7b86b8f58f7429bc8021a2e4fcc011b8f54f3198d93f5bf4fe51c76bd1bed148c0ba6e915873becf7aa6ecaa0fd1a7a7e66e1a7d388f6a3b
-  data.tar.gz: b8c68768c1a88591c7d3bcc372c283e920c6e609f3fef228d605375ddec05ed0cd5fdd52aa44a650d94518029cc629610b5cafb3b9ecd831740ee4eeda8c9be3
+  metadata.gz: 994f4d86e1c2d1aa40a173ed0b77a2fcff39cbfe1e0390c728f92887be3817e3851610f85796c24a61919c05789a718c3939ee965866a1fda2f72ed2e68492b1
+  data.tar.gz: c9de80ede2701f7e58bde7f803b473980383d6ec9ac3d8509f2fe0e74ff15a200e48025ecc2535bc7102b82f1199228806d1e1999de32745ae07123123cb7edd

data/.github/workflows/release.yaml

@@ -15,7 +15,7 @@ jobs:
           git config user.name 'Barkibot'
           git config user.email 'dev+bot@barkibu.com'
       - name: Release Gem
-        uses: discourse/publish-rubygems-action@v2
+        uses: discourse/publish-rubygems-action@v3
         env:
           GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
           RUBYGEMS_API_KEY: ${{secrets.RUBYGEMS_API_KEY}}

data/.gitignore

@@ -20,3 +20,5 @@
 /.idea
 .env
 .claude/worktrees/
+
+vendor/

data/CHANGELOG.md

@@ -6,7 +6,20 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [unreleased]
-- See diff: https://github.com/barkibu/kb-ruby/compare/v0.31.0...HEAD
+- See diff: https://github.com/barkibu/kb-ruby/compare/v1.0.0...HEAD
+
+## [1.0.0]
+- [Breaking changes] Split the single global request timeout into per-phase budgets: `KB.config.request.connect_timeout` (default 1s, bounds TCP connect + TLS handshake), `write_timeout` (default 3s), `read_timeout` (default 5s). `KB.config.request.timeout` is removed — assigning it now raises `NoMethodError` at boot. Migration: a previous global `timeout` maps to `read_timeout` (e.g. `KB_REQUEST_TIMEOUT_SECONDS=12` → `read_timeout = 12`).
+- [Breaking changes] Switch the HTTP adapter from http.rb (`faraday-http`) to Net::HTTP (`faraday-net_http`) — the stock adapter honours all three phase timeouts, aligns the KB client with the rest of our outbound HTTP, and opens a one-line upgrade path to `faraday-net_http_persistent` for connection reuse. Raw error classes change accordingly (`Net::OpenTimeout`/`Net::ReadTimeout`/`Net::WriteTimeout`/`Errno::*` instead of `HTTP::*`) — relevant to APM span queries on `error.type`.
+- Faraday-level wrapping keeps the same three classes (`Faraday::TimeoutError`/`ConnectionFailed`/`SSLError`) with one movement between them: a connect/TLS-phase expiry now surfaces as `Faraday::ConnectionFailed` (was `Faraday::TimeoutError`), making `ConnectionFailed` cleanly mean "the request never got through the pipe".
+- Net::HTTP's idempotent auto-retry stays disabled (`max_retries = 0`, enforced by the adapter and pinned by a spec) — no behaviour change vs. the previous no-retry client.
+- There is no total request budget anymore; worst-case wall clock is the sum of the phase budgets rather than a single number.
+- barkibu-kb-fake: disable Sinatra's host authorization (`set :host_authorization, permitted_hosts: []`). Sinatra >= 4.1 authorizes the Host header and, when it infers a development environment, only permits localhost-style hosts — rejecting requests to the stubbed KB host with `403 Host not permitted`. Permitting the host on the consumer side stops working with the net_http adapter, because WebMock intercepts before Net::HTTP adds the Host header, so the header is absent and can never match a permitted list. Consumers can drop their own `set :host_authorization` workarounds.
+
+## [0.32.0]
+- Allow Ruby 3.3/3.4: raise `required_ruby_version` ceiling to `< 3.6` (floor stays `>= 2.6`)
+- No runtime dependency changes — safe to take with a scoped conservative update (`bundle lock --update barkibu-kb barkibu-kb-fake --conservative`)
+- Test-env only: add `base64`/`bigdecimal` dev dependencies and an `activesupport >= 7.1` floor (suite now also validated against ActiveSupport 8). Runtime constraints for consumers unchanged.
 
 ## [0.31.0]
 - Add `KB::Pet#pet_parent`, a memoized lookup via `PetParent.find`

data/Gemfile

@@ -5,3 +5,5 @@ git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }
 # Specify your gem's dependencies in kb.gemspec
 gemspec name: 'barkibu-kb'
 gemspec name: 'barkibu-kb-fake', development_group: :fake
+
+gem 'activesupport', '>= 7.1'

data/Gemfile.lock

@@ -1,17 +1,17 @@
 PATH
   remote: .
   specs:
-    barkibu-kb (0.31.0)
+    barkibu-kb (1.0.0)
       activemodel (>= 4.0.2)
       activerecord
       activesupport (>= 3.0.0)
       dry-configurable (~> 0.9)
       faraday
-      faraday-http
+      faraday-net_http (~> 1.0)
       faraday_middleware
       i18n
-    barkibu-kb-fake (0.31.0)
-      barkibu-kb (= 0.31.0)
+    barkibu-kb-fake (1.0.0)
+      barkibu-kb (= 1.0.0)
       countries
       sinatra
       webmock
@@ -19,29 +19,40 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    activemodel (7.0.4.2)
-      activesupport (= 7.0.4.2)
-    activerecord (7.0.4.2)
-      activemodel (= 7.0.4.2)
-      activesupport (= 7.0.4.2)
-    activesupport (7.0.4.2)
-      concurrent-ruby (~> 1.0, >= 1.0.2)
+    activemodel (8.1.3)
+      activesupport (= 8.1.3)
+    activerecord (8.1.3)
+      activemodel (= 8.1.3)
+      activesupport (= 8.1.3)
+      timeout (>= 0.4.0)
+    activesupport (8.1.3)
+      base64
+      bigdecimal
+      concurrent-ruby (~> 1.0, >= 1.3.1)
+      connection_pool (>= 2.2.5)
+      drb
       i18n (>= 1.6, < 2)
+      json
+      logger (>= 1.4.2)
       minitest (>= 5.1)
-      tzinfo (~> 2.0)
+      securerandom (>= 0.3)
+      tzinfo (~> 2.0, >= 2.0.5)
+      uri (>= 0.13.1)
     addressable (2.8.0)
       public_suffix (>= 2.0.2, < 5.0)
     ast (2.4.2)
+    base64 (0.3.0)
+    bigdecimal (4.1.2)
     byebug (11.1.3)
-    concurrent-ruby (1.2.2)
+    concurrent-ruby (1.3.7)
+    connection_pool (3.0.2)
     countries (5.3.1)
       unaccent (~> 0.3)
     crack (0.4.5)
       rexml
     diff-lcs (1.4.4)
     docile (1.4.0)
-    domain_name (0.5.20190701)
-      unf (>= 0.0.5, < 1.0.0)
+    drb (2.2.3)
     dry-configurable (0.16.1)
       dry-core (~> 0.6)
       zeitwerk (~> 2.6)
@@ -63,9 +74,6 @@ GEM
     faraday-em_http (1.0.0)
     faraday-em_synchrony (1.0.0)
     faraday-excon (1.1.0)
-    faraday-http (1.1.0)
-      faraday (~> 1.0)
-      http (>= 4.0, < 6)
     faraday-httpclient (1.0.1)
     faraday-multipart (1.0.4)
       multipart-post (~> 2)
@@ -76,25 +84,14 @@ GEM
     faraday-retry (1.0.3)
     faraday_middleware (1.2.0)
       faraday (~> 1.0)
-    ffi (1.15.5)
-    ffi-compiler (1.0.1)
-      ffi (>= 1.0.0)
-      rake
     hashdiff (1.0.1)
-    http (5.1.1)
-      addressable (~> 2.8)
-      http-cookie (~> 1.0)
-      http-form_data (~> 2.2)
-      llhttp-ffi (~> 0.4.0)
-    http-cookie (1.0.5)
-      domain_name (~> 0.5)
-    http-form_data (2.3.0)
-    i18n (1.12.0)
+    i18n (1.15.2)
       concurrent-ruby (~> 1.0)
-    llhttp-ffi (0.4.0)
-      ffi-compiler (~> 1.0)
-      rake (~> 13.0)
-    minitest (5.18.0)
+    json (2.20.0)
+    logger (1.7.0)
+    minitest (6.0.6)
+      drb (~> 2.0)
+      prism (~> 1.5)
     multipart-post (2.3.0)
     mustermann (3.0.0)
       ruby2_keywords (~> 0.0.1)
@@ -102,6 +99,7 @@ GEM
     parser (3.3.0.5)
       ast (~> 2.4.1)
       racc
+    prism (1.9.0)
     public_suffix (4.0.7)
     racc (1.7.3)
     rack (2.2.6.3)
@@ -139,6 +137,7 @@ GEM
       rubocop (~> 1.19)
     ruby-progressbar (1.13.0)
     ruby2_keywords (0.0.5)
+    securerandom (0.4.1)
     simplecov (0.21.2)
       docile (~> 1.1)
       simplecov-html (~> 0.11)
@@ -151,13 +150,12 @@ GEM
       rack-protection (= 3.0.5)
       tilt (~> 2.0)
     tilt (2.1.0)
+    timeout (0.6.1)
     tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)
     unaccent (0.4.0)
-    unf (0.1.4)
-      unf_ext
-    unf_ext (0.0.8.2)
     unicode-display_width (2.5.0)
+    uri (1.1.1)
     webmock (3.14.0)
       addressable (>= 2.8.0)
       crack (>= 0.3.2)
@@ -168,8 +166,11 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
+  activesupport (>= 7.1)
   barkibu-kb!
   barkibu-kb-fake!
+  base64
+  bigdecimal
   bundler
   byebug
   rake (>= 12.3.3)

data/README.md

@@ -43,9 +43,18 @@ KB.config.log_level = :debugger # :info by default
 ```
 
 #### Request timeout configuration
+
+Timeouts are per phase, and each phase fails with its own error class:
+`connect_timeout` maps to Net::HTTP's `open_timeout`, bounding TCP connect plus
+the TLS handshake (`Net::OpenTimeout`); `read_timeout` bounds the wait for the
+response (`Net::ReadTimeout`); `write_timeout` bounds sending the request
+(`Net::WriteTimeout`).
+
 ```ruby
 # config/initializers/kb_ruby.rb
-KB.config.request.timeout = 10 # 5 by default
+KB.config.request.connect_timeout = 2 # 1 by default; TCP connect + TLS handshake
+KB.config.request.write_timeout = 4   # 3 by default
+KB.config.request.read_timeout = 10   # 5 by default
 ```
 
 ### Exposed Entities

data/barkibu-kb-fake.gemspec

@@ -32,7 +32,7 @@ Gem::Specification.new do |spec|
   spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
-  spec.required_ruby_version = '>= 2.6', '< 3.3'
+  spec.required_ruby_version = '>= 2.6', '< 3.6'
 
   spec.add_runtime_dependency 'barkibu-kb', KB::VERSION
   spec.add_runtime_dependency 'countries'

data/barkibu-kb.gemspec

@@ -34,7 +34,7 @@ Gem::Specification.new do |spec|
   spec.bindir        = 'exe'
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = ['lib']
-  spec.required_ruby_version = '>= 2.6', '< 3.3'
+  spec.required_ruby_version = '>= 2.6', '< 3.6'
 
   spec.add_dependency 'dry-configurable', '~> 0.9'
   spec.add_development_dependency 'bundler'
@@ -44,12 +44,14 @@ Gem::Specification.new do |spec|
   spec.add_development_dependency 'rubocop'
   spec.add_development_dependency 'rubocop-rspec'
   spec.add_development_dependency 'simplecov'
+  spec.add_development_dependency 'base64'
+  spec.add_development_dependency 'bigdecimal'
   spec.add_development_dependency 'webmock'
   spec.add_runtime_dependency 'activemodel', '>= 4.0.2'
   spec.add_runtime_dependency 'activerecord'
   spec.add_runtime_dependency 'activesupport', '>= 3.0.0'
   spec.add_runtime_dependency 'faraday'
-  spec.add_runtime_dependency 'faraday-http'
+  spec.add_runtime_dependency 'faraday-net_http', '~> 1.0'
   spec.add_runtime_dependency 'faraday_middleware'
   spec.add_runtime_dependency 'i18n'
 end

data/lib/barkibu-kb.rb

@@ -5,7 +5,6 @@ require 'active_support'
 require 'active_support/core_ext/array'
 require 'faraday'
 require 'faraday_middleware'
-require 'faraday/http'
 require 'dry/configurable'
 
 module KB
@@ -19,7 +18,9 @@ module KB
   setting :log_level, default: :info
 
   setting :request do
-    setting :timeout, default: 5
+    setting :connect_timeout, default: 1
+    setting :write_timeout, default: 3
+    setting :read_timeout, default: 5
   end
 end
 

data/lib/kb/client.rb

@@ -74,9 +74,7 @@ module KB
     end
 
     def connection
-      @connection ||= Faraday.new(url: base_url,
-                                  headers: headers,
-                                  request: { timeout: KB.config.request.timeout }) do |conn|
+      @connection ||= Faraday.new(url: base_url, headers: headers, request: request_timeouts) do |conn|
         conn.response :json
         conn.response :raise_error
         if KB.config.log_level == :debugger
@@ -84,8 +82,16 @@ module KB
             logger.filter(/(X-api-key:\s)("\w+")/, '\1[API_KEY_SCRUBBED]')
           end
         end
-        conn.adapter :http
+        conn.adapter :net_http
       end
     end
+
+    def request_timeouts
+      {
+        open_timeout: KB.config.request.connect_timeout,
+        write_timeout: KB.config.request.write_timeout,
+        read_timeout: KB.config.request.read_timeout
+      }
+    end
   end
 end

data/lib/kb/fake/api.rb

@@ -44,6 +44,16 @@ module KB
       include BoundedContext::PetFamily::PetContracts
       include BoundedContext::PetFamily::Products
 
+      # Sinatra >= 4.1 ships Rack::Protection::HostAuthorization, which authorizes
+      # on the Host header and, when it infers a development environment, only
+      # permits localhost-style hosts — rejecting requests addressed to the
+      # stubbed KB host with "403 Host not permitted". Permitting specific hosts
+      # on the consumer side is no fix either: with the net_http adapter WebMock
+      # intercepts before Net::HTTP adds the Host header, so the header is absent
+      # and can never match a permitted list. An empty list disables the check;
+      # it guards against DNS rebinding, which cannot apply to an in-process fake.
+      set :host_authorization, permitted_hosts: []
+
       set :state, ApiState.new
 
       def self.snapshot

data/lib/kb/version.rb

@@ -1,3 +1,3 @@
 module KB
-  VERSION = '0.31.0'.freeze
+  VERSION = '1.0.0'.freeze
 end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: barkibu-kb
 version: !ruby/object:Gem::Version
-  version: 0.31.0
+  version: 1.0.0
 platform: ruby
 authors:
 - Léo Figea
-autorequire: 
 bindir: exe
 cert_chain: []
-date: 2026-07-09 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: dry-configurable
@@ -122,6 +121,34 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: base64
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: bigdecimal
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
@@ -193,19 +220,19 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: faraday-http
+  name: faraday-net_http
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: faraday_middleware
   requirement: !ruby/object:Gem::Requirement
@@ -321,7 +348,6 @@ licenses:
 - MIT
 metadata:
   homepage_uri: https://app.barkibu.com
-post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -332,15 +358,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '2.6'
   - - "<"
     - !ruby/object:Gem::Version
-      version: '3.3'
+      version: '3.6'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key: 
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Barkibu's Knowledge Base API sdk
 test_files: []