Skip Menu |

This queue is for tickets about the Catalyst-Controller-reCAPTCHA CPAN distribution.

Report information
The Basics
Id: 62695
Status: open
Priority: 0/
Queue: Catalyst-Controller-reCAPTCHA
People
Owner: Nobody in particular
Requestors: perl [...] okko.net
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 0.7
Fixed in: (no value)

History Show all quoted text
  Thu Nov 04 11:40:31 2010 oskari.ojala [...] frantic.com - Ticket created
Subject: sub captcha_check returns true for invalid captcha, does not match documentation
The docs say the sub captcha_check would return true if there is success and claims one can do: if ( $c->forward(captcha_check) ) { # do something based on the reCAPTCHA passing } but the sub returns 1 even if the result was not valid.
  Thu Dec 29 08:28:08 2011 DIEGOK [...] cpan.org - Correspondence added
Hello!, On Thu Nov 04 11:40:31 2010, Okko wrote: Show quoted text
> The docs say the sub captcha_check would return true if there is success > and claims one can do: > > if ( $c->forward(captcha_check) ) { > # do something based on the reCAPTCHA passing > } > > > but the sub returns 1 even if the result was not valid. >
Now it's fixed (also the test is fixed). I've just pushed it to CPAN, it should be on your mirror soon :-) Thank you very much for this report. -dk
  Thu Dec 29 08:28:09 2011 The RT System itself - Status changed from 'new' to 'open'
  Thu Dec 29 08:40:25 2011 DIEGOK [...] cpan.org - Correspondence added
Sorry, I've fixed it on Catalyst::TraitFor::Controller::reCAPTCHA. It was fixed and released in this module long time ago. You should uptate to 0.8 or better move to the TraitFor one. On Thu Nov 04 11:40:31 2010, Okko wrote: Show quoted text
> The docs say the sub captcha_check would return true if there is success > and claims one can do: > > if ( $c->forward(captcha_check) ) { > # do something based on the reCAPTCHA passing > } > > > but the sub returns 1 even if the result was not valid. >