Page MenuHomePhabricator

Add "phd.user" with `sudo` hooks for SSH/HTTP writes
ClosedPublic

Authored by epriestley on Nov 15 2013, 4:49 PM.
Tags
None
Referenced Files
F13187063: D7589.diff
Sat, May 11, 4:18 AM
Unknown Object (File)
Tue, May 7, 6:07 AM
Unknown Object (File)
Fri, May 3, 5:09 AM
Unknown Object (File)
Mon, Apr 29, 3:56 PM
Unknown Object (File)
Thu, Apr 25, 11:01 PM
Unknown Object (File)
Wed, Apr 24, 10:29 PM
Unknown Object (File)
Sat, Apr 20, 6:06 PM
Unknown Object (File)
Apr 11 2024, 11:46 PM
Subscribers

Details

Reviewers
btrahan
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rP476b27d9c8b2: Add "phd.user" with `sudo` hooks for SSH/HTTP writes
Summary

Ref T2230. When fully set up, we have up to three users who all need to write into the repositories:

  • The webserver needs to write for HTTP receives.
  • The SSH user needs to write for SSH receives.
  • The daemons need to write for "git fetch", "git clone", etc.

These three users don't need to be different, but in practice they are often not likely to all be the same user. If for no other reason, making them all the same user requires you to "git clone httpd@host.com", and installs are likely to prefer "git clone git@host.com".

Using three different users also allows better privilege separation. Particularly, the daemon user can be the only user with write access to the repositories. The webserver and SSH user can accomplish their writes through sudo, with a whitelisted set of commands. This means that even if you compromise the ssh user, you need to find a way to escallate from there to the daemon user in order to, e.g., write arbitrary stuff into the repository or bypass commit hooks.

This lays some of the groundwork for a highly-separated configuration where the SSH and HTTP users have the fewest privileges possible and use sudo to interact with repositories. Some future work which might make sense:

  • Make bin/phd respect this (require start as the right user, or as root and drop privileges, if this configuration is set).
  • Execute all git/hg/svn commands via sudo?

Users aren't expected to configure this yet so I haven't written any documentation.

Test Plan

Added an SSH user ("dweller") and gave it sudo by adding this to /etc/sudoers:

dweller ALL=(epriestley) SETENV: NOPASSWD: /usr/bin/git-upload-pack, /usr/bin/git-receive-pack

Then I ran git pushes and pulls over SSH via "dweller@localhost". They successfully interacted with the repository on disk as the "epriestley" user.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

epriestley updated this revision to Unknown Object (????).Nov 15 2013, 4:49 PM

Fix a minor issue with git-receive-pack, which doesn't take a -- arg.