pusher/pusher-http-python

Tornado backend does not work with Tornado 5.x

zdexter opened this issue · 3 comments

The deprecated alias tornado.concurrent.TracebackFuture has been removed.

https://github.com/tornadoweb/tornado/blob/f6e98e279bf78d2984351062906eb100a7303c35/docs/releases/v5.0.0.rst#tornadoconcurrent

from pusher.tornado import TornadoBackend
from tornado.concurrent import TracebackFuture
ImportError: cannot import name 'TracebackFuture'

looks like the solution is to use tornado.concurrent.Future:

 .. versionchanged:: 5.0

       This class was previoiusly available under the name
       ``TracebackFuture``. This name, which was deprecated since
       version 4.0, has been removed. When `asyncio` is available
       ``tornado.concurrent.Future`` is now an alias for
       `asyncio.Future`. Like `asyncio.Future`, callbacks are now
       always scheduled on the `.IOLoop` and are never run
       synchronously.

http://www.tornadoweb.org/en/stable/_modules/tornado/concurrent.html

Thanks for reporting this. Tornado 4.0 was released over 4 years ago now. I think we can drop support for earlier versions at this point. I'll create a PR to switch to tornado.concurrent.Future.

The fix is in the latest release: v2.0.2