[PATCH] revsets: added authuri when login information is requested (issue3209)
mads at kiilerich.com
Thu Jan 16 05:47:53 CST 2014
On 01/16/2014 01:54 AM, Lucas Moscovicz wrote:
> # HG changeset patch
> # User Lucas Moscovicz <lmoscovicz at fb.com>
> # Date 1389833180 28800
> # Wed Jan 15 16:46:20 2014 -0800
> # Node ID c7b43e384d85c17c7e90326ff9c4f6d5d907c196
> # Parent f694cd81b600b65d23dcdc7a02cfd6a57dd1d018
> revsets: added authuri when login information is requested (issue3209)
It is not clear from the description or the patch what it has to do with
revsets. Reading the bug makes it more clear. I would file it under
"http" or "url" instead of "revsets". Revsets is just one of the ways to
show this "issue".
> When users are using a revset they can get multiple password prompts.
> This prompts have no extra information about which password is being requested
> so I added the authuri to the prompt to make it recognizable.
What to the prompt look like with this patch? An example would be
helpful (especially when there is no test).
Could you add some test coverage of this? It is almost covered by
test-http.t. Forcing it to interactive mode could probably make it reach
the line you are touching.
> diff --git a/mercurial/url.py b/mercurial/url.py
> --- a/mercurial/url.py
> +++ b/mercurial/url.py
> @@ -35,7 +35,8 @@
> if not self.ui.interactive():
> raise util.Abort(_('http authorization required'))
This message should get the same treatment.
> - self.ui.write(_("http authorization required\n"))
> + self.ui.write(_("http authorization required for %s\n") %
> + urllib.splitquery(authuri))
It should probably also use util.hidepassword ... just in case.
urllib.splitquery is not documented on
http://docs.python.org/2/library/urllib.html and the docstring do not
say much on how it works on urls. Based on experience with other things
realted to url/http, I wonder if the function is available and stable in
all relevant Python versions.
More information about the Mercurial-devel