joerussbowman / AsyncMongo-Sessions

A sessions library for Tornado using AsyncMongo
30 stars 2 forks source link

Not working on asynchronous methods #1

Closed joerussbowman closed 12 years ago

joerussbowman commented 13 years ago

The current implementation won't work for any asynchronous methods. It's because it calls the method and then immediately tries to save the function. I need to figure out how to wrap the method call.

joerussbowman commented 13 years ago

I've got a couple ways to skin this cat. Going to play around with a few things and try to come up with a solution that is also easiest to read/maintain.

joerussbowman commented 13 years ago

This is working, as long as you have pull request 367 on facebook/tornado implemented. Will close when that change, or something similar, makes it into tornado proper.

joerussbowman commented 12 years ago

This is fixed, no changes required to tornado. I've monkey patched what's necessary to save the data at the end of the request objects finish method.