Skip to content

Conversation

@marshall-lee
Copy link
Contributor

@jjb
Copy link

jjb commented Nov 14, 2024

@marshall-lee why is this PR for net-http-persistent and not net-http? (i didn't see any context regarding that in the linked bug, sorry if i'm missing something)

@marshall-lee
Copy link
Contributor Author

@jjb

net-http-persistent is kinda wrapper for net-http. So if you assign something via Net::HTTP::Persistent#certificate=, Net::HTTP::Persistent#ca_path=, etc — these assignments will be mirrored on Net::HTTP connection instance.

The problem is that net-http supports another such option extra_chain_cert= not having a corresponding Net::HTTP::Persistent writer for. This PRs adds extra_chain_cert= support.

@marshall-lee
Copy link
Contributor Author

ping @tenderlove

@tenderlove
Copy link
Collaborator

We need to figure out a way to automatically do this so that we don't have to change net-http-persistent every time net-http changes. 🥲

@tenderlove tenderlove merged commit 8fb912f into drbrain:master Dec 4, 2024
16 checks passed
@tenderlove
Copy link
Collaborator

Shipped this in 4.0.5

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.

3 participants