------------------------------------------------------------
revno: 13241
revision-id: squid3@treenet.co.nz-20161216120519-10g7sfp18rckdr11
parent: squid3@treenet.co.nz-20160509131121-4saxl1wj54n20lgf
fixes bug: http://bugs.squid-cache.org/show_bug.cgi?id=3290
author: Garri Djavadyan <garryd@comnet.uz>
committer: Amos Jeffries <squid3@treenet.co.nz>
branch nick: 3.4
timestamp: Sat 2016-12-17 01:05:19 +1300
message:
  Bug 3290: authenticate_ttl not working for digest authentication
------------------------------------------------------------
# Bazaar merge directive format 2 (Bazaar 0.90)
# revision_id: squid3@treenet.co.nz-20161216120519-10g7sfp18rckdr11
# target_branch: http://bzr.squid-cache.org/bzr/squid3/3.4
# testament_sha1: ee701c632d10018c28fb72ef50846795395ed95e
# timestamp: 2016-12-16 12:50:59 +0000
# source_branch: http://bzr.squid-cache.org/bzr/squid3/3.4
# base_revision_id: squid3@treenet.co.nz-20160509131121-\
#   4saxl1wj54n20lgf
# 
# Begin patch
=== modified file 'src/auth/digest/UserRequest.cc'
--- src/auth/digest/UserRequest.cc	2015-01-19 16:42:41 +0000
+++ src/auth/digest/UserRequest.cc	2016-12-16 12:05:19 +0000
@@ -166,12 +166,7 @@
     auth_user->credentials(Auth::Ok);
 
     /* password was checked and did match */
-    debugs(29, 4, HERE << "user '" << auth_user->username() << "' validated OK");
-
-    /* auth_user is now linked, we reset these values
-     * after external auth occurs anyway */
-    auth_user->expiretime = current_time.tv_sec;
-    return;
+    debugs(29, 4, "user '" << auth_user->username() << "' validated OK");
 }
 
 Auth::Direction

=== modified file 'src/auth/digest/auth_digest.cc'
--- src/auth/digest/auth_digest.cc	2015-01-19 16:42:41 +0000
+++ src/auth/digest/auth_digest.cc	2016-12-16 12:05:19 +0000
@@ -1089,6 +1089,10 @@
          * the user agent won't change user name without warning.
          */
         authDigestUserLinkNonce(digest_user, nonce);
+
+        /* auth_user is now linked, we reset these values
+         * after external auth occurs anyway */
+        auth_user->expiretime = current_time.tv_sec;
     } else {
         debugs(29, 9, HERE << "Found user '" << username << "' in the user cache as '" << auth_user << "'");
         digest_user = static_cast<Auth::Digest::User *>(auth_user.getRaw());