[Catalyst] Plugin::Authentication overrides $c->req->user

Tomas Doran bobtfish at bobtfish.net
Wed Feb 11 21:53:48 GMT 2009


On 11 Feb 2009, at 21:37, Peter Karman wrote:

> Daniel Westermann-Clark wrote on 02/11/2009 02:53 PM:
>> If no one is using this behavior, I'd be happy to provide patches to
>> deprecate or remove it.
>
> Why not just add a remote_user() method on $c->req instead? It's a
> little more typing, but is more explicit about where the value comes
> from and doesn't potentially break any existing apps.


+1 for remote_user

Catalyst::Request's user method has been marked as deprecated in the  
docs since 2006 please make calling it issue a warning once per- 
request, so that it can killed outright some time in the future.

Patches on 5.80 welcome :)

Cheers
t0m




More information about the Catalyst mailing list