switch to module::install
[catagits/Catalyst-Plugin-Session.git] / lib / Catalyst / Plugin / Session.pm
index ca267c9..08ef366 100644 (file)
@@ -11,8 +11,9 @@ use Catalyst::Exception ();
 use Digest              ();
 use overload            ();
 use Object::Signature   ();
+use Carp;
 
-our $VERSION = "0.05";
+our $VERSION = '0.20';
 
 my @session_data_accessors; # used in delete_session
 BEGIN {
@@ -22,6 +23,7 @@ BEGIN {
           _sessionid
           _session
           _session_expires
+          _extended_session_expires
           _session_data_sig
           _flash
           _flash_keep_keys
@@ -67,7 +69,7 @@ sub setup_session {
 
     %$cfg = (
         expires        => 7200,
-        verify_address => 1,
+        verify_address => 0,
         %$cfg,
     );
 
@@ -87,25 +89,51 @@ sub prepare_action {
     $c->NEXT::prepare_action(@_);
 }
 
-sub finalize {
+sub finalize_headers {
     my $c = shift;
 
+    # fix cookie before we send headers
     $c->_save_session_expires;
+
+    return $c->NEXT::finalize_headers(@_);
+}
+
+sub finalize {
+    my $c = shift;
+    my $ret = $c->NEXT::finalize(@_);
+
+    # then finish the rest
+    $c->finalize_session;
+    return $ret;
+}
+
+sub finalize_session {
+    my $c = shift;
+
+    $c->NEXT::finalize_session;
+
+    $c->_save_session_id;
     $c->_save_session;
     $c->_save_flash;
 
-    $c->NEXT::finalize(@_);
+    $c->_clear_session_instance_data;
+}
+
+sub _save_session_id {
+    my $c = shift;
+
+    # we already called set when allocating
+    # no need to tell the state plugins anything new
 }
 
 sub _save_session_expires {
     my $c = shift;
 
-    if ( defined(my $expires = $c->_session_expires) ) {
+    if ( defined($c->_session_expires) ) {
+        my $expires = $c->session_expires; # force extension
+
         my $sid = $c->sessionid;
         $c->store_session_data( "expires:$sid" => $expires );
-
-        $c->_session_expires(undef);
-        $c->_tried_loading_session_expires(undef);
     }
 }
 
@@ -122,9 +150,6 @@ sub _save_session {
             my $sid = $c->sessionid;
             $c->store_session_data( "session:$sid" => $session_data );
         }
-
-        $c->_session(undef);
-        $c->_tried_loading_session_data(undef);
     }
 }
 
@@ -149,22 +174,20 @@ sub _save_flash {
         else {
             $c->delete_session_data("flash:$sid");
         }
-        
-        $c->_flash(undef);
-        $c->_tried_loading_flash_data(undef);
     }
 }
 
 sub _load_session_expires {
     my $c = shift;
-    return if $c->_tried_loading_session_expires;
+    return $c->_session_expires if $c->_tried_loading_session_expires;
     $c->_tried_loading_session_expires(1);
 
     if ( my $sid = $c->sessionid ) {
         my $expires = $c->get_session_data("expires:$sid") || 0;
 
         if ( $expires >= time() ) {
-            return $c->extend_session_expires( $expires );
+            $c->_session_expires( $expires );
+            return $expires;
         } else {
             $c->delete_session( "session expired" );
             return 0;
@@ -176,11 +199,11 @@ sub _load_session_expires {
 
 sub _load_session {
     my $c = shift;
-    return if $c->_tried_loading_session_data;
+    return $c->_session if $c->_tried_loading_session_data;
     $c->_tried_loading_session_data(1);
 
     if ( my $sid = $c->sessionid ) {
-        if ( $c->session_expires ) {    # > 0
+        if ( $c->_load_session_expires ) {    # > 0
 
             my $session_data = $c->get_session_data("session:$sid") || return;
             $c->_session($session_data);
@@ -211,7 +234,7 @@ sub _load_session {
 
 sub _load_flash {
     my $c = shift;
-    return if $c->_tried_loading_flash_data;
+    return $c->_flash if $c->_tried_loading_flash_data;
     $c->_tried_loading_flash_data(1);
 
     if ( my $sid = $c->sessionid ) {
@@ -239,18 +262,26 @@ sub _expire_session_keys {
     }
 }
 
+sub _clear_session_instance_data {
+    my $c = shift;
+    $c->$_(undef) for @session_data_accessors;
+    $c->NEXT::_clear_session_instance_data; # allow other plugins to hook in on this
+}
+
 sub delete_session {
     my ( $c, $msg ) = @_;
 
-    # delete the session data
-    my $sid = $c->sessionid || return;
-    $c->delete_session_data("${_}:${sid}") for qw/session expires flash/;
+    $c->log->debug("Deleting session" . ( defined($msg) ? "($msg)" : '(no reason given)') ) if $c->debug;
 
-    $c->delete_session_id;
+    # delete the session data
+    if ( my $sid = $c->sessionid ) {
+        $c->delete_session_data("${_}:${sid}") for qw/session expires flash/;
+        $c->delete_session_id($sid);
+    }
 
     # reset the values in the context object
     # see the BEGIN block
-    $c->$_(undef) for @session_data_accessors;
+    $c->_clear_session_instance_data;
 
     $c->_session_delete_reason($msg);
 }
@@ -266,11 +297,10 @@ sub session_delete_reason {
 sub session_expires {
     my $c = shift;
 
-    if ( defined( my $expires = $c->_session_expires ) ) {
+    if ( defined( my $expires = $c->_extended_session_expires ) ) {
         return $expires;
     } elsif ( defined( $expires = $c->_load_session_expires ) ) {
-        $c->_session_expires($expires);
-        return $expires;
+        return $c->extend_session_expires( $expires );
     } else {
         return 0;
     }
@@ -278,7 +308,7 @@ sub session_expires {
 
 sub extend_session_expires {
     my ( $c, $expires ) = @_;
-    $c->_session_expires( my $updated = $c->calculate_extended_session_expires( $expires ) );
+    $c->_extended_session_expires( my $updated = $c->calculate_extended_session_expires( $expires ) );
     $c->extend_session_id( $c->sessionid, $updated );
     return $updated;
 }
@@ -295,7 +325,10 @@ sub calculate_extended_session_expires {
 
 sub reset_session_expires {
     my ( $c, $sid ) = @_;
-    $c->_session_expires( my $exp = $c->calculate_initial_session_expires );
+    
+    my $exp = $c->calculate_initial_session_expires;
+    $c->_session_expires( $exp );
+    $c->_extended_session_expires( $exp );
     $exp;
 }
 
@@ -312,7 +345,8 @@ sub _load_sessionid {
 
     if ( defined( my $sid = $c->get_session_id ) ) {
         if ( $c->validate_session_id($sid) ) {
-            $c->_sessionid( $sid );
+            # temporarily set the inner key, so that validation will work
+            $c->_sessionid($sid);
             return $sid;
         } else {
             my $err = "Tried to set invalid session ID '$sid'";
@@ -327,9 +361,8 @@ sub _load_sessionid {
 sub session_is_valid {
     my $c = shift;
 
-    $c->_load_session; # check expiry and also __address, etc
-
-    if ( $c->_session ) {
+    # force a check for expiry, but also __address, etc
+    if ( $c->_load_session ) {
         return 1;
     } else {
         return;
@@ -357,14 +390,39 @@ sub keep_flash {
     (@{$href}{@keys}) = ((undef) x @keys);
 }
 
-sub flash {
+sub _flash_data { 
     my $c = shift;
     $c->_flash || $c->_load_flash || do {
         $c->create_session_id_if_needed;
         $c->_flash( {} );
+    };
+}
+
+sub _set_flash {
+    my $c = shift;
+    if (@_) {
+        my $items = @_ > 1 ? {@_} : $_[0];
+        croak('flash takes a hash or hashref') unless ref $items;
+        @{ $c->_flash }{ keys %$items } = values %$items;
     }
 }
 
+sub flash {
+    my $c = shift;
+    $c->_flash_data;
+    $c->_set_flash(@_);
+    return $c->_flash;
+}
+
+sub clear_flash {
+    my $c = shift;
+    
+    #$c->delete_session_data("flash:" . $c->sessionid); # should this be in here? or delayed till finalization?
+    $c->_flash_key_hashes({});
+    $c->_flash_keep_keys({});
+    $c->_flash({});
+}
+
 sub session_expire_key {
     my ( $c, %keys ) = @_;
 
@@ -472,8 +530,7 @@ __END__
 
 =head1 NAME
 
-Catalyst::Plugin::Session - Generic Session plugin - ties together server side
-storage and client side state required to maintain session data.
+Catalyst::Plugin::Session - Generic Session plugin - ties together server side storage and client side state required to maintain session data.
 
 =head1 SYNOPSIS
 
@@ -485,11 +542,11 @@ storage and client side state required to maintain session data.
       Session::State::Cookie
       /;
 
-       # you can replace Store::FastMmap with Store::File - both have sensible
-       # default configurations (see their docs for details)
+    # you can replace Store::FastMmap with Store::File - both have sensible
+    # default configurations (see their docs for details)
 
-       # more complicated backends are available for other scenarios (DBI storage,
-       # etc)
+    # more complicated backends are available for other scenarios (DBI storage,
+    # etc)
 
 
     # after you've loaded the plugins you can save session data
@@ -534,7 +591,7 @@ made by the same client.
 
 This plugin links the two pieces together.
 
-=head1 RECCOMENDED BACKENDS
+=head1 RECOMENDED BACKENDS
 
 =over 4
 
@@ -616,9 +673,13 @@ of every request.
         }
     }
 
+=item clear_flash
+
+Zap all the keys in the flash regardless of their current state.
+
 =item keep_flash @keys
 
-If you wawnt to keep a flash key for the next request too, even if it hasn't
+If you want to keep a flash key for the next request too, even if it hasn't
 changed, call C<keep_flash> and pass in the keys as arguments.
 
 =item delete_session REASON
@@ -690,10 +751,15 @@ It's only effect is if the (off by default) C<flash_to_stash> configuration
 parameter is on - then it will copy the contents of the flash to the stash at
 prepare time.
 
+=item finalize_headers
+
+This method is extended and will extend the expiry time before sending
+the response.
+
 =item finalize
 
-This method is extended and will extend the expiry time, as well as persist the
-session data if a session exists.
+This method is extended and will call finalize_session after the other
+finalizes run.  Here we persist the session data if a session exists.
 
 =item initialize_session_data
 
@@ -728,25 +794,15 @@ Currently it returns a concatenated string which contains:
 
 =over 4
 
-=item *
-
-A counter
-
-=item *
-
-The current time
-
-=item *
+=item * A counter
 
-One value from C<rand>.
+=item * The current time
 
-=item *
+=item * One value from C<rand>.
 
-The stringified value of a newly allocated hash reference
+=item * The stringified value of a newly allocated hash reference
 
-=item *
-
-The stringified value of the Catalyst context object
+=item * The stringified value of the Catalyst context object
 
 =back
 
@@ -772,11 +828,38 @@ Or even more directly, replace C<generate_session_id>:
 Also have a look at L<Crypt::Random> and the various openssl bindings - these
 modules provide APIs for cryptographically secure random data.
 
+=item finalize_session
+
+Clean up the session during C<finalize>.
+
+This clears the various accessors after saving to the store.
+
 =item dump_these
 
 See L<Catalyst/dump_these> - ammends the session data structure to the list of
 dumped objects if session ID is defined.
 
+
+=item calculate_extended_session_expires
+
+=item calculate_initial_session_expires
+
+=item create_session_id_if_needed
+
+=item delete_session_id
+
+=item extend_session_expires
+
+=item extend_session_id
+
+=item get_session_id
+
+=item reset_session_expires
+
+=item session_is_valid
+
+=item set_session_id
+
 =back
 
 =head1 USING SESSIONS DURING PREPARE
@@ -788,16 +871,16 @@ State plugins must set $c->session ID before C<prepare_action>, and during
 C<prepare_action> L<Catalyst::Plugin::Session> will actually load the data from
 the store.
 
-       sub prepare_action {
-               my $c = shift;
+    sub prepare_action {
+        my $c = shift;
 
-               # don't touch $c->session yet!
+        # don't touch $c->session yet!
 
-               $c->NEXT::prepare_action( @_ );
+        $c->NEXT::prepare_action( @_ );
 
-               $c->session;  # this is OK
-               $c->sessionid; # this is also OK
-       }
+        $c->session;  # this is OK
+        $c->sessionid; # this is also OK
+    }
 
 =head1 CONFIGURATION
 
@@ -820,6 +903,8 @@ hours).
 When true, C<<$c->request->address>> will be checked at prepare time. If it is
 not the same as the address that initiated the session, the session is deleted.
 
+Defaults to false.
+
 =item flash_to_stash
 
 This option makes it easier to have actions behave the same whether they were
@@ -913,25 +998,21 @@ plugin should be safe enough.
 
 =head1 AUTHORS
 
-=over 4
-
-=item Andy Grundman
+Andy Grundman
 
-=item Christian Hansen
+Christian Hansen
 
-=item Yuval Kogman, C<nothingmuch@woobling.org> (current maintainer)
+Yuval Kogman, C<nothingmuch@woobling.org> (current maintainer)
 
-=item Sebastian Riedel
-
-=back
+Sebastian Riedel
 
 And countless other contributers from #catalyst. Thanks guys!
 
 =head1 COPYRIGHT & LICENSE
 
-       Copyright (c) 2005 the aforementioned authors. All rights
-       reserved. This program is free software; you can redistribute
-       it and/or modify it under the same terms as Perl itself.
+    Copyright (c) 2005 the aforementioned authors. All rights
+    reserved. This program is free software; you can redistribute
+    it and/or modify it under the same terms as Perl itself.
 
 =cut