hapijs / catbox

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

Catbox timeout runs under current domain #63

Closed kpdecker closed 10 years ago

kpdecker commented 10 years ago

The cache expiration timeout is run in the context of the current domain, meaning a reference is retained to that request in hapi environments and other environments the cache cleanup may be invalidated due to the current domain being disposed.

hueniverse commented 10 years ago

This would be a catbox-memory specific issue, but I am not convinced it needs fixing.

kpdecker commented 10 years ago

Is protect going to be updated so it does not retain the request object at least?

On Tue, May 13, 2014 at 10:53 AM, Eran Hammer notifications@github.comwrote:

Closed #63 https://github.com/spumko/catbox/issues/63.

— Reply to this email directly or view it on GitHubhttps://github.com/spumko/catbox/issues/63#event-120625521 .

hueniverse commented 10 years ago

Done. hapi 4.1.2

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.