Mason

On Tue, 25 Nov 2003, Jonathan Gardner wrote:

> There was a bug about this earlier:
> http://marc.theaimsgroup.com/?l=mason&m=103761463726621&w=2
> http://www.masonhq.com/resources/todo/view.html?id=414

This was changed long ago.

> If I write a handler thusly:
>
> sub handler {
>     my $r = Apache::Request->new(shift);
>
>     my $res;
>     {
>         $res = Session->get_session($r);
>         last unless $res == OK;
>         $res = HTML::Mason::ApacheHandler->handler($r);
>     }
>     Session->store_sessions($r);
>
>     return $res;
> }
>
> It is apparent that the '$r' in the HTML::Mason pages is absent any POST
> parameters, as Session->get_session will call $r->param.
>
> Any pointers on how to construct my handler so that both Session and Mason
> have access to all the args and uploaded arguments?

You need to use instance() as well, I think.


-dave

/*=======================
House Absolute Consulting
 http://www.houseabsolute.com
=======================*/

> On Tue, 25 Nov 2003, Jonathan Gardner wrote:
>
> > There was a bug about this earlier:
> > http://marc.theaimsgroup.com/?l=mason&m=103761463726621&w=2
> > http://www.masonhq.com/resources/todo/view.html?id=414
>
> This was changed long ago.
>
> > If I write a handler thusly:
> >
> > sub handler {
> >     my $r = Apache::Request->new(shift);
> >
> >     my $res;
> >     {
> >         $res = Session->get_session($r);
> >         last unless $res == OK;
> >         $res = HTML::Mason::ApacheHandler->handler($r);
> >     }
> >     Session->store_sessions($r);
> >
> >     return $res;
> > }
> >
> > It is apparent that the '$r' in the HTML::Mason pages is absent any POST
> > parameters, as Session->get_session will call $r->param.
> >
> > Any pointers on how to construct my handler so that both Session and
Mason
> > have access to all the args and uploaded arguments?
>
> You need to use instance() as well, I think.

Isn't the problem that $r->param reads all the POST content, leaving nothing
for Mason? Or is this an old problem? I remember this used to be the case
with $r->content.

Jon

=2D----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Tuesday 25 November 2003 19:02, Dave Rolsky wrote:
> On Tue, 25 Nov 2003, Jonathan Gardner wrote:
> > If I write a handler thusly:
> >
> > sub handler {
> >     my $r =3D Apache::Request->new(shift);
> >
> >     my $res;
> >     {
> >         $res =3D Session->get_session($r);
> >         last unless $res =3D=3D OK;
> >         $res =3D HTML::Mason::ApacheHandler->handler($r);
> >     }
> >     Session->store_sessions($r);
> >
> >     return $res;
> > }
> >
> > It is apparent that the '$r' in the HTML::Mason pages is absent any POST
> > parameters, as Session->get_session will call $r->param.
> >
> > Any pointers on how to construct my handler so that both Session and
> > Mason have access to all the args and uploaded arguments?
>
> You need to use instance() as well, I think.
>

You are absolutely correct. Here is my final handler:

sub handler {
  my $r =3D Apache::Request->instance(shift);

  my $res;
  {
    $res =3D Session->get_sessions($r);
    last if $res !=3D OK;
    $res =3D HTML::Mason::ApacheHandler->handler($r);

  }

  Session->store_sessions($r);

  return $res;
}

Thank you Dave, you just saved me a ton of debugging work.

=2D --=20
Jonathan M. Gardner
Web Developer, Amazon.com
jonagard@... - (206) 266-2906
=2D----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)

iD8DBQE/xPJUBFeYcclU5Q0RAiY4AJ9i983uy0WR02AFUiChWWao/qH6SwCg2hM0
YU0Zj92LF8m6jCTX696DiD4=3D
=3D1HtW
=2D----END PGP SIGNATURE-----

=2D----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Wednesday 26 November 2003 09:38, Jonathan Swartz wrote:
>
> Isn't the problem that $r->param reads all the POST content, leaving
> nothing for Mason? Or is this an old problem? I remember this used to be
> the case with $r->content.
>

This is true, if you one routine uses 'instance' and another 'new' to get t=
he=20
Apache::Request object. The first one to call param or upload will slurp al=
l=20
of the POST content, leaving nothing for the other one.

That is why, as Dave Rolsky pointed out, both must use 'instance'. HTML::Ma=
son=20
already uses 'instance' - this was fixed long ago. My handler used 'new'.=20
Once I changed that to 'instance', everything worked wonderfully.

=2D --=20
Jonathan M. Gardner
Web Developer, Amazon.com
jonagard@... - (206) 266-2906
=2D----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.1 (GNU/Linux)

iD8DBQE/xPLeBFeYcclU5Q0RAkD3AKDMD7EMVTOEnZi7/Dj+iyCIB3mbgQCgqquv
eip99P1TiS31hNrGmJryCos=3D
=3DO9YB
=2D----END PGP SIGNATURE-----