svn commit: r720538 - /incubator/qpid/trunk/qpid/cpp/src/qpid/broker/SaslAuthenticator.cpp

[gs...@apache.org]

Author: gsim
Date: Tue Nov 25 08:55:20 2008
New Revision: 720538

URL: http://svn.apache.org/viewvc?rev=720538&view=rev
Log:
Clean up logging a bit: only warn once, not for every connection


Modified:
    incubator/qpid/trunk/qpid/cpp/src/qpid/broker/SaslAuthenticator.cpp

Modified: incubator/qpid/trunk/qpid/cpp/src/qpid/broker/SaslAuthenticator.cpp
URL: http://svn.apache.org/viewvc/incubator/qpid/trunk/qpid/cpp/src/qpid/broker/SaslAuthenticator.cpp?rev=720538&r1=720537&r2=720538&view=diff
==============================================================================
--- incubator/qpid/trunk/qpid/cpp/src/qpid/broker/SaslAuthenticator.cpp (original)
+++ incubator/qpid/trunk/qpid/cpp/src/qpid/broker/SaslAuthenticator.cpp Tue Nov 25 08:55:20 2008
@@ -121,6 +121,7 @@
     if (c.getBroker().getOptions().auth) {
         return std::auto_ptr<SaslAuthenticator>(new CyrusAuthenticator(c));
     } else {
+        QPID_LOG(warning, "SASL: No Authentication Performed");
         return std::auto_ptr<SaslAuthenticator>(new NullAuthenticator(c));
     }
 }
@@ -136,7 +137,6 @@
 
 void NullAuthenticator::start(const string& mechanism, const string& response)
 {
-    QPID_LOG(warning, "SASL: No Authentication Performed");
     if (mechanism == "PLAIN") { // Old behavior
         if (response.size() > 0 && response[0] == (char) 0) {
             string temp = response.substr(1);