Page MenuHomePhabricator

When a client submits an overlong "sourcePath", truncate it and continue
ClosedPublic

Authored by epriestley on Aug 22 2019, 2:16 AM.
Tags
None
Referenced Files
F13273718: D20727.diff
Fri, May 31, 2:30 AM
F13269929: D20727.diff
Wed, May 29, 8:14 AM
F13242354: D20727.id49419.diff
Thu, May 23, 2:36 AM
F13233877: D20727.diff
Tue, May 21, 2:38 AM
F13230082: D20727.diff
Mon, May 20, 8:39 PM
F13212155: D20727.diff
Fri, May 17, 6:25 AM
F13195736: D20727.diff
Sun, May 12, 10:35 PM
F13179693: D20727.diff
Wed, May 8, 9:22 PM
Subscribers
None

Details

Summary

Ref T13385. Currently, if you run arc diff in a CWD with more than 255 characters, the workflow fatals against the length of the sourcePath database column.

In the long term, removing this property is likely desirable.

For now, truncate long values and continue. This only meaningfully impacts relatively obscure interactive SVN workflows negatively, and even there, "some arc commands are glitchy in very long working directories in SVN" is still better than "arc diff fatals".

Test Plan
  • Modified arc to submit very long source paths.
  • Ran arc diff.
    • Before: Fatal when inserting >255 characters into sourcePath.
    • After: Path truncated at 255 bytes.

Diff Detail

Repository
rP Phabricator
Branch
spath1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 23276
Build 31981: Run Core Tests
Build 31980: arc lint + arc unit

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Aug 22 2019, 2:28 AM
This revision was automatically updated to reflect the committed changes.