Skip to content

Allow Faraday client configuration (rebase of #138) - #152

Open
sabman wants to merge 1 commit into
masterfrom
fix/pr-138-rebase
Open

sabman wants to merge 1 commit into
masterfrom
fix/pr-138-rebase

Conversation

@sabman

@sabman sabman commented Aug 18, 2026

Copy link
Copy Markdown
Member

Summary

Original PR: #138 by @srook

Note: 4th positional arg is preserved for fidelity to the original PR; we may want to move this into http_options later.

Test plan

  • Specs pass
  • Configurer is invoked for default connections
  • Keep-alive connections still work

Based on #138 by @srook. Squashed and rebased onto current master so
the configurer runs for both one-shot and keep-alive connections.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant