Browse Source

Strip user/password from URL in error message.

Eugene V. Lyubimkin 17 years ago
parent
commit
4805f1cfd7
2 changed files with 20 additions and 1 deletions
  1. 17 1
      apt-pkg/algorithms.cc
  2. 3 0
      debian/changelog

+ 17 - 1
apt-pkg/algorithms.cc

@@ -23,6 +23,7 @@
     
 #include <apti18n.h>
 #include <sys/types.h>
+#include <regex.h>
 #include <cstdlib>
 #include <algorithm>
 #include <iostream>
@@ -1342,7 +1343,22 @@ bool ListUpdate(pkgAcquireStatus &Stat,
 
       (*I)->Finished();
 
-      _error->Warning(_("Failed to fetch %s  %s\n"),(*I)->DescURI().c_str(),
+	  // stripping username/password from URI if present
+	  string descUri = (*I)->DescURI();
+	  regex_t userPassRegex;
+	  regcomp(&userPassRegex, "\\://(\\w+)\\:(\\w+)@", REG_EXTENDED);
+	  regmatch_t userPassMatch;
+	  regexec(&userPassRegex, descUri.c_str(), 1, &userPassMatch, 0);
+	  if (userPassMatch.rm_so != -1) // regexp matched
+	  {
+         // really stripping
+		 size_t stripStart = userPassMatch.rm_so + 3;
+		 size_t stripEnd = userPassMatch.rm_eo;
+		 descUri = descUri.substr(0, stripStart) +
+            descUri.substr(stripEnd, string::npos);
+      }
+
+      _error->Warning(_("Failed to fetch %s  %s\n"), descUri.c_str(),
 	      (*I)->ErrorText.c_str());
 
       if ((*I)->Status == pkgAcquire::Item::StatTransientNetworkError) 

+ 3 - 0
debian/changelog

@@ -4,6 +4,9 @@ apt (0.7.17) unstable; urgency=low
   * apt-pkg/acquire-item.cc:
     - Added fallback to uncompressed 'Packages' if neither 'bz2' nor 'gz'
       available. (Closes: #409284)
+  * apt-pkg/algorithm.cc:
+    - Strip username and password from source URL in error message.
+      (Closes: #425150)
 
  -- Eugene V. Lyubimkin <jackyf.devel@gmail.com>  Fri, 24 Oct 2008 23:45:17 +0300