Homec4science

Enable Mercurial reads and writes over SSH

Authored by epriestley <git@epriestley.com> on Nov 11 2013, 21:18.

Description

Enable Mercurial reads and writes over SSH

Summary:
Ref T2230. This is substantially more complicated than Git, but mostly because Mercurial's protocol is a like 50 ad-hoc extensions cobbled together. Because we must decode protocol frames in order to determine if a request is read or write, 90% of this is implementing a stream parser for the protocol.

Mercurial's own parser is simpler, but relies on blocking reads. Since we don't even have methods for blocking reads right now and keeping the whole thing non-blocking is conceptually better, I made the parser nonblocking. It ends up being a lot of stuff. I made an effort to cover it reasonably well with unit tests, and to make sure we fail closed (i.e., reject requests) if there are any parts of the protocol I got wrong.

A lot of the complexity is sharable with the HTTP stuff, so it ends up being not-so-bad, just very hard to verify by inspection as clearly correct.

Test Plan:

  • Ran hg clone over SSH.
  • Ran hg fetch over SSH.
  • Ran hg push over SSH, to a read-only repo (error) and a read-write repo (success).

Reviewers: btrahan, asherkin

Reviewed By: btrahan

CC: aran

Maniphest Tasks: T2230

Differential Revision: https://secure.phabricator.com/D7553

Details

Committed
epriestley <git@epriestley.com>Nov 11 2013, 21:18
Pushed
aubortJan 31 2017, 17:16
Parents
rPHac7c73922696: Fix `--depth N` clones in Git
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH8840f60218ef: Enable Mercurial reads and writes over SSH (authored by epriestley <git@epriestley.com>).Nov 11 2013, 21:18