-
Notifications
You must be signed in to change notification settings - Fork 338
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace HTTPoison with Hackney #100
Comments
@DavidAntaramian are you tracking the v2 branch here that @dmvt pushed up yesterday? This would be a good point for that rewrite. @dmvt we should ideally have a journey issue now to support an RFC process for this and to serve as a nexus point for related discussion/issues. |
@DavidAntaramian @joshsmith Working on the RFC now and generally support this idea. Stay tuned... |
Let's continue this discussion over at #101 |
Conversation has moved to #102. |
HTTPoison doesn't offer any benefits over using Hackney directly, and it just adds another dependency to pin against; often times the use of HTTPoison will obfuscate the lower level support provided by Hackney. I would recommend swapping out HTTPoison for Hackney, which is a very simple process. I'm willing to do it if we're in agreement.
The text was updated successfully, but these errors were encountered: