Fix spelling test, split init method out of constructor for easier sub-classing
Tomas Doran [Sun, 14 Sep 2008 19:35:31 +0000 (19:35 +0000)]
Changes
lib/Catalyst/Authentication/Credential/HTTP.pm
t/04pod_spelling.t

diff --git a/Changes b/Changes
index d143e28..2029ae4 100644 (file)
--- a/Changes
+++ b/Changes
@@ -1,7 +1,7 @@
 1.005  2008-09-XX
   - Added better description of the use of the domains and
     use_uri_for options (info culled from RFC2617).
-  - 
+  - Split init method out to make this module easier to sub-class.
 
 1.004  2008-09-12
    - Add tests for use_uri_for configuration options.
index 8067185..d4912a5 100644 (file)
@@ -23,12 +23,17 @@ sub new {
     
     $self->realm($realm);
     
+    $self->init;
+    return $self;
+}    
+
+sub init {
+    my ($self) = @_;
     my $type = $self->_config->{'type'} ||= 'any';
     
     if (!grep /$type/, ('basic', 'digest', 'any')) {
         Catalyst::Exception->throw(__PACKAGE__ . " used with unsupported authentication type: " . $type);
     }
-    return $self;
 }
 
 sub authenticate {
@@ -455,6 +460,10 @@ C<get_digest_authorization_nonce> methods as shown below.
 
 Simple constructor.
 
+=item init
+
+Validates that $config is ok.
+
 =item authenticate $c, $realm, \%auth_info
 
 Tries to authenticate the user, and if that fails calls
index 5188c23..d9cdb5d 100644 (file)
@@ -21,3 +21,8 @@ Kogman
 Yuval
 auth
 username
+Authorization
+authorization
+sess
+init
+ok