Skip to content

Fix dynamic headers for Net::HTTP requests - #120

Open
Guflly wants to merge 1 commit into
ashkan18:masterfrom
Guflly:fix/dynamic-http-headers
Open

Guflly wants to merge 1 commit into
ashkan18:masterfrom
Guflly:fix/dynamic-http-headers

Conversation

@Guflly

@Guflly Guflly commented Aug 3, 2026

Copy link
Copy Markdown

Fixes #58.

This makes the Net::HTTP adapter merge headers from the current request context, matching the Faraday adapter. Headers set through client.options[:headers] now take effect on later requests.

Tests:

  • 97 specs
  • RuboCop on the changed Ruby files

@Guflly
Guflly force-pushed the fix/dynamic-http-headers branch from 2d7d7af to 5273285 Compare August 3, 2026 00:32
@dblock
dblock requested a review from rellampec August 3, 2026 20:58
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.

Dynamically change headers

1 participant