Bug #30599 for Catalyst-Plugin-Authentication-Credential-HTTP: Plugin Assumes text/html and body

Thu Nov 08 22:19:59 2007 claco [...] cpan.org - Ticket created

Subject:

Plugin Assumes text/html and body

When using this plugin with things like Catalyst::Action::REST, calling authorization_required_response assumes that the output is always text/html and that it should set body to 'Authenticaiton Required, confusing REST, which normally sets content-type to the request content-type and body to the empty rest entity for the given content-type. Imho, this plugin should just get/set auth headers, and not futz with body/content-type at all. Those aren't really the concerns of this plugin.

Wed Jan 09 04:00:56 2008 sjn [...] cpan.org - Correspondence added

From:

sjn [...] cpan.org

On Tor. 08. Nov. 2007 22:19:59, CLACO wrote: Show quoted text

> > Imho, this plugin should just get/set auth headers, and not futz with > body/content-type at all. Those aren't really the concerns of this > plugin.

FWIW, I agree. :) BTW, is the author awake?

Wed Jan 09 04:00:59 2008 The RT System itself - Status changed from 'new' to 'open'

Wed Jan 09 10:26:14 2008 claco [...] chrislaco.com - Correspondence added

Subject:	Re: [rt.cpan.org #30599] Plugin Assumes text/html and body
Date:	Wed, 09 Jan 2008 10:24:01 -0500
To:	bug-Catalyst-Plugin-Authentication-Credential-HTTP [...] rt.cpan.org
From:	"Christopher H. Laco" <claco [...] chrislaco.com>

Salve J. Nilsen via RT wrote: Show quoted text

> <URL: http://rt.cpan.org/Ticket/Display.html?id=30599 > > > On Tor. 08. Nov. 2007 22:19:59, CLACO wrote:

>> Imho, this plugin should just get/set auth headers, and not futz with >> body/content-type at all. Those aren't really the concerns of this >> plugin.

> > FWIW, I agree. :) > > BTW, is the author awake? > >

I think the author is Jess...castaway on irc. Maybe prod the Catalyst mailing list... -=Chris

Download signature.asc
application/pgp-signature 187b

Message body not shown because it is not plain text.

Sat Jul 12 07:52:38 2008 bobtfish [...] bobtfish.net - Broken in 0.09 added

Thu Sep 11 10:15:50 2008 bobtfish [...] bobtfish.net - Cc sjn added

Thu Sep 11 10:19:11 2008 bobtfish [...] bobtfish.net - Taken

Thu Sep 11 10:51:58 2008 bobtfish [...] bobtfish.net - Correspondence added

This issue has been fixed in the new Catalyst::Authentication::Credential::HTTP (from 1.003 onwards), and this module is deprecated. Therefore I'm going to resolve this bug. Please test this at your leisure, and feel free to re-raise the bug against that module if it doesn't do what you want.

Thu Sep 11 10:52:36 2008 bobtfish [...] bobtfish.net - Status changed from 'open' to 'resolved'