AttributeError: '_NullFuture' object has no attribute 'add_done_callback' about tornado HOT 5 OPEN

Hmm, I've never seen that and I'm not quite sure how you'd get there. Are you overriding RequestHandler._execute somehow?

from tornado.

What you've written is unnecessarily complicated and, not to mention, results in error.

Instead of patching RequestHandler._execute method, you should instead perform the authentication checks in the RequestHandler.prepare method of the base class.

from tornado.

Hi,
Thank you for comment, the codes are not solid I know that, I am new in the project, I wanna figure out the issue, can you refactor any part for me?

from tornado.

On the file provided by @dryalcinmehmet, there is an options method decorated with tornado.gen.coroutine, no yields just returns. Should that the be source of the _NullFuture?

from tornado.

there is an options method decorated with tornado.gen.coroutine, no yields just returns. Should that the be source of the _NullFuture?

No, tornado.gen.coroutine should still use a real future in this case.

I don't see exactly where the _NullFuture is coming from, but this interceptor decorator is problematic in a number of ways. Wrapping _execute without actually calling the underlying _execute method is probably going to break something, as is discarding the return value of the real _execute method.

I agree with @bhch that you should not be overriding _execute here and should do your auth checks in prepare() instead. We previously kinda-supported overriding _execute because prepare couldn't be asynchronous, but now that we support coroutines in prepare there's no reason to override _execute any more and you should stay away from it.

from tornado.