Bug #46274 for Catalyst-Runtime: Catalyst::Request - "$req->user" replaced with $req->remote

Thu May 21 10:42:04 2009 kmx [...] volny.cz - Ticket created

Subject:	Catalyst::Request - documentation is wrong about "$req->user"
Date:	Thu, 21 May 2009 16:41:37 +0200
To:	bug-Catalyst-Runtime [...] rt.cpan.org
From:	kmx <kmx [...] volny.cz>

Hi, As I was told on IRC today it is not true that: "$req->user Returns the currently logged in user. Deprecated. The method recommended for newer plugins is $c->user." In fact it is not deprecated and returns the user currently logged on WWW server level (e.g. Apache). It would be nice to fix the doc. -- kmx

Thu May 21 14:51:51 2009 bobtfish [...] bobtfish.net - Correspondence added

On Thu May 21 10:42:04 2009, kmx@volny.cz wrote: Show quoted text

> Hi, > > As I was told on IRC today it is not true that: > > "$req->user Returns the currently logged in user. Deprecated. The method > recommended for newer plugins is $c->user." > > In fact it is not deprecated and returns the user currently logged on > WWW server level (e.g. Apache). > > It would be nice to fix the doc.

Unfortunately, the situation is more complex than this. If you load C::Plugin::Authentication and successfully authenticate a user, then $c->req->user will be *replaced* with the user object. There have been patches floating about to make a $c->req->remote_user (which would never be changed by anything), which I applied today in r10218 I'd recommend you install the current trunk, and use this - as I'll be deprecating and removing $c->req->user with its overloaded meaning at some point in the future. I'll keep this ticket open until $c->req->remote_user is officially available in a real release. Cheers t0m

Thu May 21 14:51:51 2009 The RT System itself - Status changed from 'new' to 'open'

Thu May 21 14:51:52 2009 bobtfish [...] bobtfish.net - Given to BOBTFISH

Thu May 21 14:52:57 2009 bobtfish [...] bobtfish.net - Subject changed from 'Catalyst::Request - documentation is wrong about "$req->user"' to 'Catalyst::Request - "$req->user" replaced with $req->remote_user'

Thu May 21 14:52:57 2009 bobtfish [...] bobtfish.net - Severity Normal added

Thu May 21 14:52:57 2009 bobtfish [...] bobtfish.net - Broken in 5.80004 added

Sat Jun 27 10:29:36 2009 rafl [...] debian.org - Status changed from 'open' to 'resolved'

Sat Jun 27 10:29:57 2009 rafl [...] debian.org - Fixed in 5.80005 added

Bug #46274 for Catalyst-Runtime: Catalyst::Request - "$req->user" replaced with $req->remote_user