slack-rs / slack-rs-api

Rust interface for the Slack Web API
Apache License 2.0
109 stars 66 forks source link

Upgrade reqwest dependency #82

Closed briprowe closed 4 years ago

briprowe commented 4 years ago

The default reqwest client in newer versions of reqwest have asynchronous behavior.

Solution: Since blocking behavior is assumed by the rest of this crate, it's simplest to use reqwest's blocking client.

dten commented 4 years ago

You must use the required version (~0.9.6) or implement your own SlackWebRequestSender

dten commented 4 years ago

Thanks for the changes though. I was waiting for non alpha versions of reqwest and tungstenite before looking at async changes

sts10 commented 4 years ago

Looks like reqwest just came out of alpha for async stuff with v0.10.0

Change default Client API to async. The previous blocking client API is [available] at reqwest::blocking.

dten commented 4 years ago

Typical 🤣