Patch: Allow Mercurial (hg) scmtools back end to utilize repository credentials when accessing HTTP/HTTPS paths

Review Request #448 — Created July 10, 2008 and submitted

Information

Review Board SVN (deprecated)
536

Reviewers

A relatively simple change that seems to work ... that said, I don't really know Python at all, so it's rather likely that there's something "not quite right" here.
Tested on Debian and Windows, Python 2.5.2. It also seems to work alright with unauthenticated remote servers, though that has only been tested lightly.
chipx86
  1. 
      
  2. trunk/reviewboard/scmtools/hg.py (Diff revision 1)
     
     
     
     
     
    What if the username and/or password are blank? Maybe we should only do this if one or both are set.
    1. I thought that would be problematic as well, but it didn't complain on the add_password call when they were blank in my testing. For unauthenticated repositories, the password manager is never consulted, so it appears to be harmless.
  3. 
      
david
  1. In that case, this looks good.  Committed as r1401.  Thanks!
  2.