QPID-976: Prompt for password if it is required and not specified through settings.

git-svn-id: https://svn.apache.org/repos/asf/qpid/trunk@732702 13f79535-47bb-0310-9956-ffa450edef68

1 files changed, 21 insertions, 7 deletions

diff --git a/qpid/cpp/src/qpid/client/SaslFactory.cpp b/qpid/cpp/src/qpid/client/SaslFactory.cpp
index af18c445d8..ade301648c 100644
--- a/qpid/cpp/src/qpid/client/SaslFactory.cpp
+++ b/qpid/cpp/src/qpid/client/SaslFactory.cpp
@@ -150,8 +150,13 @@ CyrusSasl::CyrusSasl(const ConnectionSettings& s) : conn(0), settings(s)
     }
 
     callbacks[i].id = SASL_CB_PASS;
-    callbacks[i].proc = (CallbackProc*) &getPasswordFromSettings;
-    callbacks[i++].context = &settings;
+    if (settings.password.empty()) {
+        callbacks[i].proc = 0;
+        callbacks[i++].context = 0;        
+    } else {
+        callbacks[i].proc = (CallbackProc*) &getPasswordFromSettings;
+        callbacks[i++].context = &settings;
+    }
 
     callbacks[i].id = SASL_CB_LIST_END;
     callbacks[i].proc = 0;
@@ -263,13 +268,22 @@ std::string CyrusSasl::getMechanism()
 
 void CyrusSasl::interact(sasl_interact_t* client_interact)
 {
-    std::cout << "[" << client_interact->id << "] " << client_interact->challenge << " " << client_interact->prompt;
-    if (client_interact->defresult) std::cout << " (" << client_interact->defresult << ")";
-    std::cout << std::endl;
-    if (std::cin >> input) {
+
+    if (client_interact->id == SASL_CB_PASS) {
+        char* password = getpass(client_interact->prompt);
+        input = std::string(password);
         client_interact->result = input.data();
         client_interact->len = input.size();
-    }    
+    } else {
+        std::cout << client_interact->prompt;
+        if (client_interact->defresult) std::cout << " (" << client_interact->defresult << ")";
+        std::cout << ": ";
+        if (std::cin >> input) {
+            client_interact->result = input.data();
+            client_interact->len = input.size();
+        }
+    }
+
 }
 
 std::auto_ptr<SecurityLayer> CyrusSasl::getSecurityLayer(uint16_t maxFrameSize)