kallithea Changeset - 73f3ed6a91d9

Changeset - 73f3ed6a91d9

Parent rev.

Child rev.

[Not reviewed]

default

0 2 0

Mads Kiilerich (mads) - 5 years ago 2021-01-19 20:24:31
mads@kiilerich.com

ssh: handle case where hg serve_forever doesn't serve forever

serve_forever will no longer exit in Mercurial 5.7 .
This changed with https://mercurial-scm.org/repo/hg/rev/b7b8a1538161 .

2 files changed with 2 insertions and 2 deletions:

kallithea/bin/kallithea_cli_ssh.py

kallithea/lib/vcs/ssh/base.py

0 comments (0 inline, 0 general)

kallithea/bin/kallithea_cli_ssh.py

➞

Show inline comments

@@ @@ -69,7 +69,7 @@ def ssh_serve(user_id, key_id): @@
         vcs_handler = VcsHandler.make(ssh_command_parts)
         if vcs_handler is not None:
             vcs_handler.serve(user_id, key_id, client_ip)
-            assert False # serve is written so it never will terminate
+            sys.exit(0)
     sys.stderr.write("This account can only be used for repository access. SSH command %r is not supported.\n" % ssh_original_command)
     sys.exit(1)

kallithea/lib/vcs/ssh/base.py

➞

Show inline comments

@@ @@ -96,7 +96,7 @@ class BaseSshHandler(object): @@
         # on Git, or through the 'outgoing' Mercurial hook) is hardcoded to
         # ignore the environment action and always use 'pull'.
         set_hook_environment(self.authuser.username, client_ip, self.repo_name, self.vcs_type, 'push')
-        return self._serve()
         self._serve()
     def _serve(self):
         """Serve the native protocol for repository access."""

0 comments (0 inline, 0 general)