Page MenuHomePhabricator
Differential D15794

Include directory-ownership note in `sshd` setup instructions
ClosedPublic
Actions

Authored by epriestley on Apr 25 2016, 12:40 PM.
Tags
None
Referenced Files
F14357331: D15794.diff
Fri, Dec 20, 4:00 AM
Unknown Object (File)
Sat, Dec 14, 10:14 PM
Unknown Object (File)
Tue, Dec 10, 9:57 AM
Unknown Object (File)
Sat, Dec 7, 3:49 PM
Unknown Object (File)
Thu, Dec 5, 11:00 AM
Unknown Object (File)
Wed, Dec 4, 6:38 AM
Unknown Object (File)
Fri, Nov 29, 1:26 PM
Unknown Object (File)
Mon, Nov 25, 1:25 AM
View All Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T9560: dirname(phabricator-ssh-hook.sh) needs to be owned by root too
Commits
rP623ed1f4349b: Include directory-ownership note in `sshd` setup instructions
Summary

Fixes T9560. We suggest a root-owned location, but users who choose their own location instead can run into trouble.

Test Plan
  • Changed parent directory to have an non-root owner, verified that ssh no longer worked.
  • Changed parent directory back to a root owner, verified ssh worked again.

Diff Detail

Repository
rP Phabricator
Branch
sshdoc1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 11916
Build 14975: Run Core Tests
Build 14974: arc lint + arc unit

Event Timeline

epriestley updated this revision to Diff 38042.Apr 25 2016, 12:40 PM
epriestley retitled this revision from to Include directory-ownership note in `sshd` setup instructions.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
epriestley added a task: T9560: dirname(phabricator-ssh-hook.sh) needs to be owned by root too.
epriestley mentioned this in T9560: dirname(phabricator-ssh-hook.sh) needs to be owned by root too.Apr 25 2016, 12:42 PM
chad accepted this revision.Apr 25 2016, 3:04 PM
chad edited edge metadata.
This revision is now accepted and ready to land.Apr 25 2016, 3:04 PM
Closed by commit rP623ed1f4349b: Include directory-ownership note in `sshd` setup instructions (authored by epriestley, committed by epriestley). · Explain WhyApr 25 2016, 3:16 PM
This revision was automatically updated to reflect the committed changes.
epriestley mentioned this in T10751: Make Phabricator Highly Available.Apr 26 2016, 3:18 AM

Revision Contents
Changeset List

PathSize
src/
docs/
user/
userguide/
12 lines

Diff 38042

View Options

src/docs/user/userguide/diffusion_hosting.diviner

Loading...
Phabricator · Built by Phacility