search

hapijs / catbox

Multi-strategy object caching service
Other
494 stars 72 forks source link

Make callbacks in memory store real async #46

Closed cayasso closed 11 years ago

cayasso commented 11 years ago

For example in https://github.com/spumko/catbox/blob/master/lib/memory.js#L90 instead of:

return callback(null, result);

use:

return process.nextTick(function(){
    callback(null, result);
});

This will ensure your callback is async.

Best,

JB

cayasso commented 11 years ago

Hey I know you guys a very busy but any updates on this? I submitted a pull request that address this issue.

Best, JB

geek commented 11 years ago

Is this causing any issues not being async?

lock[bot] commented 4 years ago

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.