Conversation
Updating faraday to version 2.0
|
I was able to do the testing with command Note: For the unit test, I was getting same 135 failures before as well after the upgrades during the running of the unit test ( |
…onent Replaced the URI.escape method with URI.encode_www_form_component as URI.escape method is obsolete.
…ect without a block
|
are there plans to merge this? |
|
These are very similar changes to what was done on my own fork. However, I don't think anybody is really actively maintaining this Gem any more. |
|
citusdata is a Microsoft organization maintaining a fork https://github.com/citusdata/azure-storage-ruby/tree/faraday2 unfortunately when I tried looking into whether the citus team could take over maintaining this gem the email thread went nowhere |
|
We had to create our own forks in the GitHub org and take the changes as Microsoft is not maintaining this repo anymore. I would be happy if they accept this PR, at-least for the short term it will unblock many folks. Alternatively, folks can use the below below forked repo. |
|
Thanks for this! To fix the broken tests:
There are some deprecation warnings about the use of |
🖼️ Context
Upgrading the
Faradaygem from version1.0.0 to 2.0.fix #227
✅ Approach
faradaygem to version 2.0 from 1.0, I need to remove thefaraday_middlewareas it has been deprecated based on its docs and need to use the middlewarefaraday-follow_redirects. Details are present in the awesome-faraday docs.Faradayupgrade docs, all the adapters, except for thenet_http, have already been moved out and released as separate gems. So I had to add adapterfaraday-net_http_persistent(gem) as part of this upgrade.