Call store_digest_authorization_nonce() instead of $c->cache->set()
Ronald J Kimball [Tue, 14 Dec 2010 19:20:53 +0000 (19:20 +0000)]
in authenticate_digest() (RT#63669)

Changes
lib/Catalyst/Authentication/Credential/HTTP.pm

diff --git a/Changes b/Changes
index 28521be..c22f75e 100644 (file)
--- a/Changes
+++ b/Changes
@@ -1,3 +1,5 @@
+   - Call store_digest_authorization_nonce() instead of $c->cache->set()
+     in authenticate_digest() (RT#63669) (rjk)
    - Make Test::Exception and Test::MockObject be listed as test_requires
      rather than requires (RT#54029)
    - Skip WWW::Mech tests unless WWW::Mech is a new enough version (RT#55303).
index 66db13c..5897b57 100644 (file)
@@ -183,8 +183,8 @@ sub authenticate_digest {
                     $A2_digest );
             my $rq_digest = Digest::MD5::md5_hex($digest_in);
             $nonce->nonce_count($nonce_count);
-            $c->cache->set( __PACKAGE__ . '::opaque:' . $nonce->opaque,
-                $nonce );
+            my $key = __PACKAGE__ . '::opaque:' . $nonce->opaque;
+            $self->store_digest_authorization_nonce( $c, $key, $nonce );
             if ($rq_digest eq $res{response}) {
                 return $user_obj;
             }