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
Unknown Object (File)
Sun, Jun 1, 2:19 PM
Unknown Object (File)
Sat, May 31, 8:17 PM
Unknown Object (File)
May 27 2025, 11:05 AM
Unknown Object (File)
May 5 2025, 6:10 PM
Unknown Object (File)
Apr 29 2025, 8:23 PM
Unknown Object (File)
Mar 30 2025, 10:17 PM
Unknown Object (File)
Mar 20 2025, 9:56 AM
Unknown Object (File)
Mar 18 2025, 8:05 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