[Catalyst-dev] Catalyst::Plugin::Session::Store::File patch to use the Cache::FileCache expiry mechanism

[BUCHMULLER Norbert]

Hi,

here goes the patch that I promised on #dbix-class on December 15. It
makes use of the Cache:FileCache expiry mechanism to handle session
expiry, and thus delete_expired_sessions() works. (And, as a side-effect,
halves the number of cache files: previously "session:id" and
"expires:id" were stored as separate cache entries, now only "session:id"
is stored with the value belonging to "expires:id" as expiry date.)

I followed the techniques used the Catalyst::Plugin::Session::Store::DBI
module, I hope that that module is considered to be OK. (As there are
some marginal cases - eg. what happens when "expires:id" is set, but
"session:id" is not yet set.)

Please review my patch (and apply if you accept it).

norbi
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Catalyst-Plugin-Session-Store-File-use_cache_expiry_mechanism.diff.patch
Type: text/x-patch
Size: 3543 bytes
Desc: not available
Url : http://lists.scsys.co.uk/pipermail/catalyst-dev/attachments/20090120/3a3101ad/Catalyst-Plugin-Session-Store-File-use_cache_expiry_mechanism.diff.bin