Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Replace reqwest with hyper+rustls #7829

Closed
tomusdrw opened this issue Feb 7, 2018 · 0 comments
Closed

Replace reqwest with hyper+rustls #7829

tomusdrw opened this issue Feb 7, 2018 · 0 comments
Labels
F6-refactor 📚 Code needs refactoring. M4-core ⛓ Core client code / Rust. P5-sometimesoon 🌲 Issue is worth doing soon. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow.
Milestone
1.11 Prosperity

Comments

@tomusdrw
Copy link
Collaborator

tomusdrw commented Feb 7, 2018

  • Get rid of openssl dependency on non-windows, non-mac systems.
  • Get rid of threadpool for fetch requests in util/fetch (since we can use hyper async directly)
  • Possibly also rewrite miner/price_info.rs using util/fetch to get rid of old hyper dependency.
@tomusdrw tomusdrw added F6-refactor 📚 Code needs refactoring. P5-sometimesoon 🌲 Issue is worth doing soon. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow. M4-core ⛓ Core client code / Rust. labels Feb 7, 2018
@5chdn 5chdn mentioned this issue Feb 7, 2018
Merged
@5chdn 5chdn added this to the 1.10 milestone Feb 18, 2018
@kirushik kirushik mentioned this issue Feb 20, 2018
Closed
@5chdn 5chdn modified the milestones: 1.10, 1.11 Mar 1, 2018
@twittner twittner self-assigned this Mar 1, 2018
This was referenced Mar 5, 2018
Merged
Closed
@twittner twittner mentioned this issue Mar 13, 2018
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
F6-refactor 📚 Code needs refactoring. M4-core ⛓ Core client code / Rust. P5-sometimesoon 🌲 Issue is worth doing soon. Q2-easy 💃 Can be fixed by copy and pasting from StackOverflow.
Projects
None yet
Milestone
1.11 Prosperity
Development

No branches or pull requests
3 participants
@twittner @tomusdrw @5chdn