Skip Menu |

This queue is for tickets about the Catalyst-Plugin-Authentication CPAN distribution.

Report information
The Basics
Id: 50466
Status: resolved
Priority: 0/
Queue: Catalyst-Plugin-Authentication

Owner: bobtfish [...]
Requestors: tleffler [...]

Bug Information
Severity: Normal
Broken in: 0.10015
Fixed in: (no value)

Subject: Using C::A::C::Remote with Catalyst 5.7014
Download (untitled) / with headers
text/plain 490b
I am maintaining and adding functionality to a legacy Catalyst application based on Runtime version 5.7014. Until I'm able to upgrade, I've added the C::A::C::Remote module to my local lib to accommodate user login that is external to the Catalyst app. Catalyst Runtime 5.7014 does not support $c->engine->env(), so I patched the ::Remote module and attached the patch. This may be useful for others who also would like this module to be compatible with older Catalyst Runtimes. Cheers.
Subject: Remote.patch
Download Remote.patch
text/x-diff 550b
--- /tmp/orig/lib/Catalyst/Authentication/Credential/ 2009-08-27 16:00:42.000000000 -0700 +++ /tmp/new/lib/Catalyst/Authentication/Credential/ 2009-10-13 15:06:49.000000000 -0700 @@ -45,7 +45,7 @@ my $remuser; if ($self->source eq "REMOTE_USER") { # compatibility hack: - if (defined($c->engine->env)) { + if ($c->engine->can('env') && defined($c->engine->env)) { # BEWARE: $c->engine->env was broken prior 5.80005 $remuser = $c->engine->env->{REMOTE_USER}; }
Download (untitled) / with headers
text/plain 399b
Applied as r11525, thanks for the patch. It should be noted that Catalyst 5.7 is now end of life, and support / backwards compatibility patches are unlikely to be taken / welcome for much longer. Do you have any specific reasons for not being able to upgrade to Catalyst 5.80 yet? If so, please report these as bugs also, unless they're specifically documented in Catalyst::Upgrading? Cheers t0m

This service is sponsored and maintained by Best Practical Solutions and runs on infrastructure.

Please report any issues with to