[sipxtapi] r1410 committed - Fixed bug introduced some time ago with stale=true handling in authent...

[codesite...@google.com]

Revision: 1410
Author: jaroslavl1
Date: Mon Nov 29 14:19:35 2010
Log: Fixed bug introduced some time ago with stale=true handling in
authentication. When stale=true, we don't care if we tried to authenticate
before. Otherwise we attempt it only twice. This prevents repeated
authentication attempts when credentials are wrong.
http://code.google.com/p/sipxtapi/source/detail?r=1410

Modified:
/branches/sipxtapi-3.2.0/sipXtackLib/src/net/SipUserAgent.cpp

=======================================
--- /branches/sipxtapi-3.2.0/sipXtackLib/src/net/SipUserAgent.cpp Sun Jan
11 09:11:06 2009
+++ /branches/sipxtapi-3.2.0/sipXtackLib/src/net/SipUserAgent.cpp Mon Nov
29 14:19:35 2010
@@ -4509,9 +4509,10 @@
// According to RFC2617: "The server should only set stale to TRUE
// if it receives a request for which the nonce is invalid but with a
// valid digest for that nonce" - so there shouldn't be infinite
auth loop
- if (stale.compareTo("TRUE", UtlString::ignoreCase) == 0)
+ if (stale.compareTo("TRUE", UtlString::ignoreCase) != 0)
{
// Check to see if we already tried to send the credentials
+ // if stale=true is present, we always act as if we never tried
to authenticate
alreadyTriedOnce = request.hasDigestAuthorizationData(realm,
authorizationEntity);
}