[Catalyst-commits] r8473 - in Catalyst-Authentication-Credential-OpenID: . 0.09 0.09/lib/Catalyst/Authentication/Credential

[apv at dev.catalyst.perl.org]

Author: apv
Date: 2008-09-29 18:36:35 +0100 (Mon, 29 Sep 2008)
New Revision: 8473

Added:
   Catalyst-Authentication-Credential-OpenID/0.09/
Modified:
   Catalyst-Authentication-Credential-OpenID/0.09/Changes
   Catalyst-Authentication-Credential-OpenID/0.09/README
   Catalyst-Authentication-Credential-OpenID/0.09/lib/Catalyst/Authentication/Credential/OpenID.pm
Log:
This is patch for http://rt.cpan.org/Public/Bug/Display.html?id=39688
and 0.09 on its way to CPAN.



Copied: Catalyst-Authentication-Credential-OpenID/0.09 (from rev 8472, Catalyst-Authentication-Credential-OpenID/0.08)

Modified: Catalyst-Authentication-Credential-OpenID/0.09/Changes
===================================================================
--- Catalyst-Authentication-Credential-OpenID/0.08/Changes	2008-09-29 15:23:28 UTC (rev 8472)
+++ Catalyst-Authentication-Credential-OpenID/0.09/Changes	2008-09-29 17:36:35 UTC (rev 8473)
@@ -1,5 +1,8 @@
 Revision history for Catalyst::Authentication::Credential::OpenID
 
+0.09  Mon Sep 29 18:29:59 PDT 2008
+      - Bug patch for boneheaded return v detach bug given by M. Blom.
+
 0.08  Sat Jul  5 13:13:14 PDT 2008
       - Again, only change is to Makefile.PL reqs.
 

Modified: Catalyst-Authentication-Credential-OpenID/0.09/README
===================================================================
--- Catalyst-Authentication-Credential-OpenID/0.08/README	2008-09-29 15:23:28 UTC (rev 8472)
+++ Catalyst-Authentication-Credential-OpenID/0.09/README	2008-09-29 17:36:35 UTC (rev 8473)
@@ -14,16 +14,3 @@
 	make test
 	make install
 
-It is recommended that you try to install with TEST_HTTP set to a true
-value in your environment. If it is set, the tests will run a forking
-test server which acts as an OpenID provider and consumer to run tests
-within a test Catalyst application. Failure here doesn't guarantee the
-module won't work for you but success pretty much guarantees it will
-work.
-
-  # bash
-  export TEST_HTTP=1
-
-  # tcsh and friends
-  setenv TEST_HTTP 1
-

Modified: Catalyst-Authentication-Credential-OpenID/0.09/lib/Catalyst/Authentication/Credential/OpenID.pm
===================================================================
--- Catalyst-Authentication-Credential-OpenID/0.08/lib/Catalyst/Authentication/Credential/OpenID.pm	2008-09-29 15:23:28 UTC (rev 8472)
+++ Catalyst-Authentication-Credential-OpenID/0.09/lib/Catalyst/Authentication/Credential/OpenID.pm	2008-09-29 17:36:35 UTC (rev 8473)
@@ -8,7 +8,7 @@
     __PACKAGE__->mk_accessors(qw/ _config realm debug secret /);
 }
 
-our $VERSION = "0.08";
+our $VERSION = "0.09";
 
 use Net::OpenID::Consumer;
 use Catalyst::Exception ();
@@ -78,7 +78,7 @@
             delayed_return => 1,
         );
         $c->res->redirect($check_url);
-        return;
+        $c->detach();
     }
     elsif ( $c->req->params->{'openid-check'} )
     {
@@ -128,7 +128,7 @@
 
 =head1 VERSION
 
-0.08
+0.09
 
 =head1 SYNOPSIS