[Catalyst-commits] r8989 - in trunk/Catalyst-Plugin-RequireSSL: . lib/Catalyst/Plugin

[t0m at dev.catalyst.perl.org]

Author: t0m
Date: 2009-01-02 12:27:55 +0000 (Fri, 02 Jan 2009)
New Revision: 8989

Modified:
   trunk/Catalyst-Plugin-RequireSSL/Changes
   trunk/Catalyst-Plugin-RequireSSL/lib/Catalyst/Plugin/RequireSSL.pm
Log:
Fix POD Coverage and perl critic tabs test.

Modified: trunk/Catalyst-Plugin-RequireSSL/Changes
===================================================================
--- trunk/Catalyst-Plugin-RequireSSL/Changes	2009-01-01 22:14:42 UTC (rev 8988)
+++ trunk/Catalyst-Plugin-RequireSSL/Changes	2009-01-02 12:27:55 UTC (rev 8989)
@@ -1,5 +1,9 @@
 Revision history for Perl extension Catalyst::Plugin::RequireSSL
 
+0.07
+        - Fix Perl Critic test for hard tabs (t0m)
+        - Fix POD coverage (t0m)
+
 0.06    2007-03-06 11:00:00
         - Added no_cache config option to support wildcard SSL certificates.
           (Simon Elliott)

Modified: trunk/Catalyst-Plugin-RequireSSL/lib/Catalyst/Plugin/RequireSSL.pm
===================================================================
--- trunk/Catalyst-Plugin-RequireSSL/lib/Catalyst/Plugin/RequireSSL.pm	2009-01-01 22:14:42 UTC (rev 8988)
+++ trunk/Catalyst-Plugin-RequireSSL/lib/Catalyst/Plugin/RequireSSL.pm	2009-01-02 12:27:55 UTC (rev 8989)
@@ -4,7 +4,7 @@
 use base qw/Class::Accessor::Fast/;
 use NEXT;
 
-our $VERSION = '0.06';
+our $VERSION = '0.07';
 
 __PACKAGE__->mk_accessors( qw/_require_ssl _ssl_strip_output/ );
 
@@ -103,10 +103,10 @@
         $redir .= '?' . join( '&', @params );
     }  
           
-	if ( $c->config->{require_ssl}->{no_cache} ) {		
-	    delete $c->config->{require_ssl}->{$type};
-	}
-	
+    if ( $c->config->{require_ssl}->{no_cache} ) {    	
+        delete $c->config->{require_ssl}->{$type};
+    }
+    
     return $redir;
 }
 
@@ -127,7 +127,7 @@
         https => 'secure.mydomain.com',
         http => 'www.mydomain.com',
         remain_in_ssl => 0,
-		no_cache => 0,
+        no_cache => 0,
     };
 
     # in any controller methods that should be secured
@@ -171,7 +171,7 @@
 page, you can set this option to 1.  By default, this option is disabled and
 users will be redirected back to non-SSL mode as soon as possible.
 
-	no_cache 
+    no_cache 
 
 If you have a wildcard certificate you will need to set this option if you are
 using multiple domains on one instance of Catalyst.
@@ -187,6 +187,14 @@
 The browser will be redirected to the same path on your SSL server.  POST
 requests are never redirected.
 
+=head2 setup
+
+Disables this plugin if running under an engine which does not support SSL.
+
+=head2 finalize
+
+Performs the redirect to SSL url if required.
+
 =head1 KNOWN ISSUES
 
 When viewing an SSL-required page that uses static files served from the