djm / python-scrapyd-api

A Python wrapper for working with Scrapyd's API.
BSD 2-Clause "Simplified" License
268 stars 32 forks source link

Need to add "timeout" param into the get or post function #11

Closed BruceDone closed 6 years ago

BruceDone commented 7 years ago

Hi djm: Since scrapyd server may be down , so while you using this scrapyd api , you may be block by the http error or other error, need to add the timeout param to handle this situation.

tlinhart commented 7 years ago

I would appreciate this as well. I might prepare PR which would basically:

What do you think?

djm commented 6 years ago

@tlinhart It's been a while but yes, that sounds good. I would accept a PR with that interface.

tlinhart commented 6 years ago

Thanks @djm for merging!

djm commented 6 years ago

No worries, thanks for proposing and enacting it!

I'll do the PyPI release this evening and update this thread.

djm commented 6 years ago

That's out now, v2.1.0.

Thanks again :)

tlinhart commented 6 years ago

Thank you too, @djm . Just wanted to ask how documentation gets updated. Changes are not reflected there.

djm commented 6 years ago

@tlinhart Thanks for the reminder. I've fixed that + converted the base docs to Markdown as PyPI Warehouse finally supports it 🎉.