Page MenuHomePhabricator

Improve the quality of SSH error messages
ClosedPublic

Authored by epriestley on Jun 16 2020, 3:46 PM.
Tags
None
Referenced Files
F13183889: D21362.diff
Fri, May 10, 1:00 PM
Unknown Object (File)
Tue, May 7, 5:09 AM
Unknown Object (File)
Thu, May 2, 11:08 PM
Unknown Object (File)
Mon, Apr 29, 2:01 PM
Unknown Object (File)
Wed, Apr 24, 10:06 PM
Unknown Object (File)
Sun, Apr 21, 4:14 PM
Unknown Object (File)
Wed, Apr 17, 3:17 PM
Unknown Object (File)
Thu, Apr 11, 7:25 AM
Subscribers
None

Details

Summary

See PHI1784. Currently, users who pass an invalid SSH command to Phabricator's SSH handler get an unhelpful error message. Make it more helpful.

Test Plan

Ran ./bin/ssh-exec with no arguments (old, helpful error), invalid arguments (before: unhelpful error; after: helpful error), and valid arguments (old, helpful behavior).

Diff Detail

Repository
rP Phabricator
Branch
ssh1
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 24667
Build 34013: Run Core Tests
Build 34012: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Jun 16 2020, 3:59 PM
This revision was automatically updated to reflect the committed changes.