Clone repo from remote URL fails

Mads Kiilerich mads at kiilerich.com
Tue Dec 11 11:19:21 UTC 2018


On 12/11/18 10:13 AM, Thomas De Schampheleire wrote:
> kallithea_1  |   File "./kallithea/lib/vcs/backends/hg/repository.py",
> line 295, in _check_url
> kallithea_1  |     sshpeer(repoui or ui.ui(), url).lookup('tip')
> kallithea_1  | TypeError: __init__() takes at least 8 arguments (3 given)
> kallithea_1  | 2018-12-11 09:05:07.596 INFO
> [kallithea.controllers.admin.repos] Invalid repository URL


One of the annoying `except Exception` :-(


> It's been a while that I tested this, but I wonder if it could be
> related to a mercurial version? This instance is running 4.7.2.
> In commit 03dfcbe52906 there has been a change in this area, but I
> didn't investigate it further.


Yes, that change was insufficient.

Proposed fix on 
https://kallithea-scm.org/repos/kallithea-incoming/changeset/8107a49198fbbb1474f84cc0e6c8c1cfea5d2de4

Extra annoying that it just fails in a URL validation check. The actual 
clone works ... and it would fail if the URL was invalid. So it would 
perhaps be better to drop the check and just try it, similar to how ssh 
URLs are handled?

Also, it would be nice to have test coverage for this ;-)


/Mads




More information about the kallithea-general mailing list