Skip Menu |

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

Report information
The Basics
Id: 44459
Status: rejected
Priority: 0/
Queue: Catalyst-Plugin-Authentication

People
Owner: bobtfish [...] bobtfish.net
Requestors: HORSHACK [...] cpan.org
Cc:
AdminCc:

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



Subject: Documentation wrong,
Thanks for your efforts in documenting this! I found a bug in http://search.cpan.org/~bobtfish/Catalyst-Plugin-Authentication-0.10011/lib/Catalyst/Plugin/Authentication.pm In the chapter "Example" I read == __PACKAGE__->config->{'Plugin::Authentication'} = { default => { credential => { class => 'Password', password_field => 'password', password_type => 'clear' }, store => { class => 'Minimal', users => { bob => { password => "s00p3r", editor => 'yes', roles => [qw/edit delete/], }, william => { password => "s3cr3t", roles => [qw/comment/], } } } } }; == but it should be == __PACKAGE__->config->{'Plugin::Authentication'} = { default_realm => 'default', realms => { default => { credential => { class => 'Password', password_field => 'password', password_type => 'clear' }, store => { class => 'Minimal', users => { bob => { password => "s00p3r", editor => 'yes', roles => [qw/edit delete/], }, william => { password => "s3cr3t", roles => [qw/comment/], } } } } } }; == Without the key "realms" not any realm can be found when executing the code. A little later there is an example code: == $c->res->body( "hello " . $c->user->get("name") ); == where $c->user->get("name") --> undef. I think it should read == $c->res->body( "hello " . $c->user->get("id") ); == Greetings, Horshack
On Sat Mar 21 13:19:30 2009, HORSHACK wrote: Show quoted text
> Thanks for your efforts in documenting this!
<snip> Show quoted text
> Without the key "realms" not any realm can be found when executing the > code.
From the changelog: 0.10008 2008-10-23 - Updating config to allow for inclusion of realm ref's in the main config hash rather than in a subref called 'realms' I'm guessing the version you're running is below 0.10008, but you're looking at the docs on search.cpan.org for the latest version. Cheers t0m
CC: HORSHACK [...] cpan.org
Subject: Re: [rt.cpan.org #44459] Documentation wrong,
Date: Sun, 22 Mar 2009 10:18:22 +0100
To: bug-Catalyst-Plugin-Authentication [...] rt.cpan.org
From: Richard Lippmann <horshack [...] lisa.franken.de>
You are right. Thanks Tom. Greetings, Richard Tomas Doran via RT schrieb: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=44459 > > > On Sat Mar 21 13:19:30 2009, HORSHACK wrote:
>> Thanks for your efforts in documenting this!
> > <snip> >
>> Without the key "realms" not any realm can be found when executing the >> code.
> > From the changelog: > > 0.10008 2008-10-23 > - Updating config to allow for inclusion of realm ref's in the main > config hash rather than in a subref called 'realms' > > I'm guessing the version you're running is below 0.10008, but you're > looking at the docs on search.cpan.org for the latest version. > > Cheers > t0m >
Subject: Re: [rt.cpan.org #44459] Documentation wrong,
Date: Sun, 22 Mar 2009 10:23:45 +0100
To: bug-Catalyst-Plugin-Authentication [...] rt.cpan.org
From: Richard Lippmann <lippmann [...] findus-internet-opac.de>
You are right. I had an old version of the module installed and was looking in cpan where the current version is documented. Thanks, Richard Lippmann Tomas Doran via RT schrieb: Show quoted text
> <URL: https://rt.cpan.org/Ticket/Display.html?id=44459 > > > On Sat Mar 21 13:19:30 2009, HORSHACK wrote:
>> Thanks for your efforts in documenting this!
> > <snip> >
>> Without the key "realms" not any realm can be found when executing the >> code.
> > From the changelog: > > 0.10008 2008-10-23 > - Updating config to allow for inclusion of realm ref's in the main > config hash rather than in a subref called 'realms' > > I'm guessing the version you're running is below 0.10008, but you're > looking at the docs on search.cpan.org for the latest version. > > Cheers > t0m >