[freeside-commits] branch master updated. 8d51b40b5bf90351bd013ad3daa9569b2f399a3d

[Ivan]

The branch, master has been updated
       via  8d51b40b5bf90351bd013ad3daa9569b2f399a3d (commit)
      from  0b25e1048d9d41cb92da42b99e0982910539f320 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 8d51b40b5bf90351bd013ad3daa9569b2f399a3d
Author: Ivan Kohler <ivan at freeside.biz>
Date:   Wed Feb 12 11:58:52 2014 -0800

    fix login with an existing session cookie, RT#25533

diff --git a/fs_selfservice/FS-SelfService/cgi/selfservice.cgi b/fs_selfservice/FS-SelfService/cgi/selfservice.cgi
index f84f5d6..84998bb 100755
--- a/fs_selfservice/FS-SelfService/cgi/selfservice.cgi
+++ b/fs_selfservice/FS-SelfService/cgi/selfservice.cgi
@@ -101,6 +101,8 @@ if ( $cgi->param('action') =~ /^(\w+)$/ ) {
   }
 }
 
+warn $action;
+
 unless ( $nologin_actions{$action} ) {
 
   my %cookies = CGI::Cookie->fetch;
@@ -1062,7 +1064,7 @@ sub do_template {
   my $timeout = $access_info->{'timeout'} || '60';
   my $cookie = CGI::Cookie->new('-name'     => 'session',
                                 '-value'    => $session_id,
-                                '-expires'  => '+'.$timeout,
+                                '-expires'  => '+'.$timeout.'s',
                                 #'-secure'   => 1, # would be a good idea...
                                );
   if ( $name eq 'logout' ) {

-----------------------------------------------------------------------

Summary of changes:
 fs_selfservice/FS-SelfService/cgi/selfservice.cgi |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)