Feature #881

* Output of URIs (during merge, pull, push) now omits passwords.

src/Uri.cpp

@@ -189,6 +189,30 @@ bool Uri::expand (const std::string& configPrefix )
   return false;
 }
 
+////////////////////////////////////////////////////////////////////////////////
+std::string Uri::ToString ()
+{
+  if (!_parsed)
+    return std::string ();
+
+  std::string result;
+  // strip password from _user
+  std::string::size_type pos = _user.find (":");
+  result = _protocol + "://";
+
+  if (_user.length () > 0)
+    result += _user.substr (0, pos) + "@";
+  
+  result += _host;
+  
+  if (_port.length () > 0)
+    result += + ":" + _port;
+  
+  result += "/" + _path;
+
+  return result;
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 void Uri::parse ()
 {

src/Uri.h

@@ -51,6 +51,7 @@ public:
   std::string name () const;
   std::string parent () const;
   std::string extension () const;
+  std::string ToString();
   bool is_directory () const;
   bool is_local () const;
   bool append (const std::string&);

src/commands/CmdMerge.cpp

@@ -97,7 +97,7 @@ int CmdMerge::execute (std::string& output)
     if (tmpfile != "")
       remove (tmpfile.c_str ());
 
-    if (((sAutopush == "ask") && (confirm (format (STRING_CMD_MERGE_CONFIRM, uri._data))))
+    if (((sAutopush == "ask") && (confirm (format (STRING_CMD_MERGE_CONFIRM, uri.ToString ()))))
        || (bAutopush))
     {
       // Derive autopush uri from merge.default.uri? otherwise: change prompt above

src/commands/CmdPull.cpp

@@ -112,7 +112,7 @@ int CmdPull::execute (std::string& output)
       }
 		}
 
-    output += format (STRING_CMD_PULL_TRANSFERRED, uri._data) + "\n";
+    output += format (STRING_CMD_PULL_TRANSFERRED, uri.ToString ()) + "\n";
   }
   else
     throw std::string (STRING_CMD_PULL_NO_URI);

src/commands/CmdPush.cpp

@@ -96,7 +96,7 @@ int CmdPush::execute (std::string& output)
       ofile3 << ifile3.rdbuf();
 		}
 
-    output += format (STRING_CMD_PUSH_TRANSFERRED, uri._data) + "\n";
+    output += format (STRING_CMD_PUSH_TRANSFERRED, uri.ToString ()) + "\n";
   }
   else
     throw std::string (STRING_CMD_PUSH_NO_URI);