Skip Menu |

This queue is for tickets about the Catalyst-Engine-Apache CPAN distribution.

Report information
The Basics
Id: 61706
Status: resolved
Priority: 0/
Queue: Catalyst-Engine-Apache

People
Owner: bobtfish [...] bobtfish.net
Requestors: tleffler [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: 1.12
Fixed in: (no value)



Subject: Populate remote_user
C::E::Apache does populates Catalyst::Request->user, but not remote_user. As of Catalyst-Runtime 5.80005 (C::R->remote_user as added), while using the Apache engine and the Remote authN credential modules together, this leads to a code path where $r->can('remote_user) is true, but always returns 'undef'. Attached is a patch that gets C::E::Apache to populate remote_user. My environment is: - Catalyst-Engine-Apache 1.12 - RHEL 5.4 - perl 5.8.8 - Catalyst-Runtime 5.80022 - Catalyst-Plugin-Authentication 0.10016
Subject: remote_user.diff
diff -uNr Catalyst-Engine-Apache-1.12.fixtests/lib/Catalyst/Engine/Apache.pm Catalyst-Engine-Apache-1.12.remote_user/lib/Catalyst/Engine/Apache.pm --- Catalyst-Engine-Apache-1.12.fixtests/lib/Catalyst/Engine/Apache.pm 2010-09-23 16:40:46.000000000 -0700 +++ Catalyst-Engine-Apache-1.12.remote_user/lib/Catalyst/Engine/Apache.pm 2010-09-27 11:15:24.000000000 -0700 @@ -47,6 +47,7 @@ $c->request->hostname( $self->apache->connection->remote_host ); $c->request->protocol( $self->apache->protocol ); $c->request->user( $self->apache->user ); + $c->request->remote_user( $self->apache->user ); # when config options are set, check them here first if ($INC{'Apache2/ModSSL.pm'}) {
Thanks for the patch, applied as r13614