norbi.lists at nix
Jan 19, 2009, 4:26 PM
Views: 867
Permalink
|
|
Catalyst::Plugin::Session::Store::File patch to use the Cache::FileCache expiry mechanism
|
|
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
|
Catalyst-Plugin-Session-Store-File-use_cache_expiry_mechanism.diff.patch
Catalyst::Plugin::Session::Store::File patch to use the Cache::FileCache expiry mechanism
