Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

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

Report information
The Basics
Id: 67693
Status: resolved
Priority: 0/
Queue: Mojolicious-Plugin-Authentication

People
Owner: Nobody in particular
Requestors: andre [...] haruhi.org.ua
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: v0.5.0



Subject: Bugs in Mojolicious::Plugin::Authentication module
Date: Fri, 22 Apr 2011 13:30:05 +0300
To: bug-Mojolicious-Plugin-Authentication [...] rt.cpan.org
From: Andre Parker <andre [...] haruhi.org.ua>
Hi. Some issues in subj. When registering before_dispatch hook, sub receive plugins instance as first argument and controller instance as second. Using '$self->stash' producing error, because there's no method 'stash' in 'Mojolicious::Plugins' object. In 'authenticate' helper there is sub call '$self->stash->{$our_stash_key}->{validate_user}->($self, $user, $pass)', but nobody created 'validate_user' in stash. Some lines below 'load_user' called as $args->{load_user}. In same helper 'load_user' called as '{load_user}->($uid)', instead of '{load_user}->($self, $uid)'. Thx. -- Best regards, Parker Andre.
Hi Andre, Thanks for the report, and you're right they're in the 0.03 (or 0.3.0 .. damn qv() for messing up) version, I've already fixed the bugs in my local version which will be on CPAN soon(tm) as 0.5.0. The bugs you reported are all a result of coding while sleepy so I'll try not to do that anymore :D Anyway, couple hours and there'll be a proper fixed version on CPAN :)
Bugs are fixed, the code has now also been made so it doesn't require MojoX::Session, and it's on it's way to CPAN now. Thanks again :)
Subject: Re: [rt.cpan.org #67693] Bugs in Mojolicious::Plugin::Authentication module
Date: Fri, 22 Apr 2011 19:39:30 +0300
To: bug-Mojolicious-Plugin-Authentication [...] rt.cpan.org
From: Andre Parker <andre [...] haruhi.org.ua>
Hi. On Fri, Apr 22, 2011 at 3:14 PM, Ben van Staveren via RT <bug-Mojolicious-Plugin-Authentication@rt.cpan.org> wrote: Show quoted text
> Bugs are fixed, the code has now also been made so it doesn't require > MojoX::Session, and it's on it's way to CPAN now. Thanks again :)
Much better now. Thx. :) -- Best regards, Parker Andre.