in reply to Unable to create session, add params, and redirect

Two things occur to me: 1) Anonymous Monk's sample shows the header looking about right but without status. The status might matter (30#) so you could try adding a flag for it. The HTTP spec requires absolute URIs for redirect locations. You're using a relative one. While relative ones work in practice most of the time you should make it absolute.

Also, I'm not positive any of this addresses your problem but have at it- 

use strict;
use warnings;
use CGI ();
use CGI::Session();
use URI ();

my $session = CGI::Session->new
    or die CGI::Session->errstr;

my $uri = URI->new( CGI::url() );
$uri->query("page=1");
# might want to $uri->fragment(""), etc...

print $session->header(-location => $uri,
                       -status => 302 );

__END__
moo@cow[3505]~/bin>curl -i http://localhost/cgi/pm-764431-2.cgi
HTTP/1.1 302 Found
Date: Sun, 17 May 2009 05:20:44 GMT
Server: Apache/2.2.3 (Unix) mod_fastcgi/2.4.2
Set-Cookie: CGISESSID=0ea36d11385258e787e3b31a31f23cb1; path=/
Location: http://localhost/cgi/pm-764431-2.cgi?page=1
Content-Length: 0
Content-Type: text/html; charset=ISO-8859-1

[download]

Use URI::QueryParam for non-trivial query string manipulation.

Replies are listed 'Best First'.
Re^2: Unable to create session, add params, and redirect
by bradcathey (Prior) on May 17, 2009 at 12:18 UTC

    Thanks for your thinking on this. Appreciate it much. BTW, I'd never heard of URI, however, still not quite sure why you used it :)

    The only thing I don't see in your code is where you add key/value param to the session. That's what is throwing me. I need to create the new session, add a param, and then redirect. It's too many headers.

    —Brad
    "The important work of moving the world forward does not wait to be done by perfect men." George Eliot
[reply]
      Too many headers? You know that you're supposed to print headers only once.
[reply]

      Yeah, I'm sorry. I didn't read the original flow right. AM is right and I alluded to it above. You can't print headers twice or you'll get wonky behavior. Give something like this a spin. 

      my $current_page = shift || die "Need a current page";

my $session = CGI::Session->new
    or die CGI::Session->errstr;

my @pages = $session->param('authorized_pages');

push @pages, $current_page
    unless $pages[-1] eq $current_page; # Just a guess.

$session->param( authorized_pages => \@pages );
$session->flush();

my $uri = URI->new( CGI::url() );
$uri->query("page=1");
$uri->fragment("");

print $session->header(-location => $uri,
                       -status => 302 );

      [download]

      URI is in there to get the absolute URI and mess with the query string and fragment. Munging URI as strings is like parsing HTML with regexes. It can be done but it's error prone and why shoot yourself in the foot when there are great tools that are just as easy, if not easier.

[reply]
[d/l]
        You can't print headers twice or you'll get wonky behavior
        Everything after headers is content, no wonky :D
[reply]

In Section Seekers of Perl Wisdom