Skip to content

Conversation

@saiqulhaq
Copy link
Contributor

Faraday 2.11.0 has new SSL option: ciphers

@saiqulhaq saiqulhaq force-pushed the feat/ciphers-opt-support branch from 1c1a7d7 to 1dccb9e Compare September 9, 2024 00:11
@saiqulhaq
Copy link
Contributor Author

image

@saiqulhaq
Copy link
Contributor Author

This PR is to fix one of these tasks

@olleolleolle
Copy link
Member

A detail with this is: the gem will no longer after this change support Faraday 1.

Perhaps we can make it support both versions? A conditional checking for the Faraday version before trying to set the ciphers setting? Something, so that we don't have to maintain a branch with a Faraday 1.x version.

@saiqulhaq
Copy link
Contributor Author

OK let me try to make it work with Faraday v1 and v2

@saiqulhaq
Copy link
Contributor Author

RSpec with Ruby v3.3
image

RSpec with Ruby v2.6, Faraday 2.8
image

RSpec with Ruby v2.6, Faraday 1.10

  spec.add_dependency 'faraday', '~> 1.10.3'

root@41de205654e6:/app# bundle show faraday
/bundle/gems/faraday-1.10.3

image

there is an issue with Faraday v1
I had to add gem 'multipart-parser', '~> 0.1.1' to Gemfile manually

I want to add Appraisal, but it's hard since too many combinations of the Ruby and Faraday versions need to be supported.
Even the Rubocop spec is failing in the CI.

FYI, I created a new gem to switch between Ruby versions easily without bloating up our machine -> https://github.com/saiqulhaq/gemdock
https://saiqulhaq.id/very-fast-ruby-gem-development-testing

@olleolleolle
Copy link
Member

Ha, I failed to fix the Gemfile right, I _removed_a newline, which is now a lint failure. Sorry!

Gemfile:16:37: C: [Correctable] Layout/TrailingEmptyLines: Final newline missing. (https://rubystyle.guide#newline-eof)
gem 'rubocop-performance', '~> 1.20'

@saiqulhaq
Copy link
Contributor Author

All checks have passed now @olleolleolle

@olleolleolle
Copy link
Member

@saiqulhaq Can you make a rebase on top of latest changes, and force-push this branch? "This branch cannot be rebased due to conflicts" it says in the GitHub web interface, now.

@saiqulhaq saiqulhaq force-pushed the feat/ciphers-opt-support branch from 86633e9 to f40eb65 Compare October 23, 2024 09:12
@saiqulhaq saiqulhaq force-pushed the feat/ciphers-opt-support branch from 559d2e6 to b0722ed Compare August 6, 2025 03:18
@olleolleolle
Copy link
Member

Awesome, are there any tests that cover this new option, either in this repo or in the shared tests?

Thanks for sticking with it!

@saiqulhaq saiqulhaq force-pushed the feat/ciphers-opt-support branch 3 times, most recently from 425c540 to f3b6942 Compare October 7, 2025 15:59
@saiqulhaq saiqulhaq force-pushed the feat/ciphers-opt-support branch from f3b6942 to 1648f0c Compare October 7, 2025 16:14
@saiqulhaq
Copy link
Contributor Author

I have updated the README file and added RSpec tests to cover the new ciphers option.

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.

2 participants