Incorrect behavior for timeout parameter in aiohttp.request method

[asvetlov]

Now the library uses timeout only for reading data from opened socket, not for connecting stage.
So if the server don't answer I'll get a error like TimeoutError: [Errno 110] Connect call failed ('66.221.212.5', 8888) where TimeoutError is not asyncio.TimeoutError but exception derived from OSError, it's raised by OS.
Moreover, event if we put connection stage under the hood of timeout we still have a bit strange behavior: timeout will work only for connecting and reading HTTP headers but not for getting HTTP body.

I guess just to remove timeout parameter from the API and recommend to use asyncio.wait_for directly if needed.

Thoughts?

[fafhrd91]

i agree, timeout is redundant. let is remove it.

[lock]

This thread has been automatically locked since there has not been
any recent activity after it was closed. Please open a new issue for
related bugs.

If you feel like there's important points made in this discussion,
please include those exceprts into that new issue.