Skip to content

net_http adapter: Backport the code of #38 to version 1.0.1#39

Merged
olleolleolle merged 1 commit into
lostisland:1-0-stablefrom
ma2gedev:fix/backport-pr-38-into-1.x
Jul 24, 2024
Merged

net_http adapter: Backport the code of #38 to version 1.0.1#39
olleolleolle merged 1 commit into
lostisland:1-0-stablefrom
ma2gedev:fix/backport-pr-38-into-1.x

Conversation

@ma2gedev

Copy link
Copy Markdown
Contributor

This PR is the backport of #38 to version v1.0.1 .

Could you create a branch for version 1.0.1 (something like 1.x)? I've temporarily set the target branch to main, but I will change the target branch of the PR to the new one afterwards.

`env[:ssl]` is an instance of `Faraday::SSLOptions`, and as a result,
`configure_ssl` is always executed, even when the connection is to HTTP.
Fixes lostisland#37
@olleolleolle

Copy link
Copy Markdown
Member

@ma2gedev I will make a 1-0-stable branch, where https://github.com/lostisland/faraday-net_http/tree/v1.0.1 is.

@olleolleolle olleolleolle changed the base branch from main to 1-0-stable July 24, 2024 08:33
@olleolleolle

Copy link
Copy Markdown
Member

@ma2gedev Now, there exists a branch.

@ma2gedev

Copy link
Copy Markdown
Contributor Author

@olleolleolle Thank you so much! ❤️

@olleolleolle olleolleolle merged commit 9d1abe5 into lostisland:1-0-stable Jul 24, 2024
@ma2gedev ma2gedev deleted the fix/backport-pr-38-into-1.x branch July 24, 2024 08:42
@olleolleolle

Copy link
Copy Markdown
Member

@ma2gedev In order to get to a release of 1.0.2, there was a little oil change needed. But now, I think any subsequent 1-0-stable release can be done using our automation.

https://rubygems.org/gems/faraday-net_http/versions/1.0.2 now exists.

@ma2gedev

Copy link
Copy Markdown
Contributor Author

@olleolleolle Thank you for your work about both of 1.0.2 and 3.1.1. ❤️

@olleolleolle

Copy link
Copy Markdown
Member

No, thank you for suggesting it!

@dpsi

dpsi commented Jul 29, 2024

Copy link
Copy Markdown

thank you everyone involved here, as someone who is stuck on faraday v1 still, it is much appreciated!

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