Issues/67

[komuw]

Thank you for contributing to naz.
Every contribution to naz is important.

Contributor offers to license certain software (a “Contribution” or multiple
“Contributions”) to naz, and naz agrees to accept said Contributions,
under the terms of the MIT License.
Contributor understands and agrees that naz shall have the irrevocable and perpetual right to make
and distribute copies of any Contribution, as well as to create and distribute collective works and
derivative works of any Contribution, under the MIT License.

Now,

What(What have you changed?)

• Introduce a configurable timeout when trying to connect to SMSC
• Try and detect when the connection to SMSC is disconnected and attempt to re-connect & re-bind
• bugfix; asyncio.streams.StreamWriter.drain should not be called concurrently by multiple coroutines[1]
• when shutting down, naz now tries to make sure that write buffers are properly flushed[2][3]

Why(Why did you change it?)

• Make naz more failure tolerant
• Fixes: Connection lost on await writer.drain() #67
• Fixes: drain should not be called by multiple coroutines #114
• Fixes: add timeouts to connections, reading & writing sockets etc #116
• Fixes: naz.Client.shutdown should call client.writer.drain #117
• Fixes: detect closed StreamWriter/StreamReader not working #120

References:

1. https://bugs.python.org/issue29930
2. https://vorpus.org/blog/some-thoughts-on-asynchronous-api-design-in-a-post-asyncawait-world/
3. Graceful shutdown can (I think) truncate connections that it shouldn't aio-libs/aiohttp#1369

[komuw]

thanks to the work(blogposts etc) of @njsmith otherwise the fixes in this PR would not have been possible