Fix a SSH URL parsing issue breaking private GitHub repositories.

Review Request #2014 — Created Dec. 25, 2010 and submitted

Information

Review Board

Reviewers

Fix a SSH URL parsing issue breaking private GitHub repositories.

The code to parse a SSH-based URL to extract the hostname and username
was giving precedence to the username provided to the function, rather than
that in the URL. In the case of private GitHub repositories, the server
needed to be accessed with the "git" user, but we were using the
repository owner's username instead.

We now give precedence to the URL, using the provided username as the
fallback. This fixes adding private GitHub repositories (so long as
the Review Board instance has a DSA or RSA keypair and the public key is
added to the GitHub repository).
I was able to add a private GitHub repository on new Review Board server
with a fresh RSA keypair added to GitHub.
chipx86
  1. I accidentally already committed this, but well, take a look anyway and make sure I'm sane :)
  2. 
      
david
  1. Ship It!
  2. 
      
Loading...