Medium / medium-wordpress-plugin

The official WordPress plugin for cross-posting to Medium.
https://medium.com
Apache License 2.0
211 stars 40 forks source link

Bumping the request timeout to 20. #63

Closed huckphin closed 8 years ago

huckphin commented 8 years ago

Hello @mikkot, @majelbstoat, @kylehg,

Please review the following commits I made in branch 'chad-bump-timeout'.

4ea91332377cf481c7d743a8e82f739c726617d9 (2016-02-03 14:33:10 -0800) Bumping the request timeout to 20.

R=@mikkot R=@majelbstoat R=@kylehg

mikkot commented 8 years ago

LG - timeouts cause interruptions and duplicates, and given the low volume of ongoing migrations (still), I don't think the impact of increasing the timeout will be noticeable on medium.com

kylehg commented 8 years ago

I'm a bit skeptical that this will do much—I suspect if it's gonna time out at 10 seconds it'd time out at 20, and I thought that Sepia already cuts off requests at 10?—but safe to try. LG

huckphin commented 8 years ago

Yeah, it does take a bit to download the associated media and images, which takes some time.