Page MenuHomePhabricator

Fix two minor issues with "arc download"
ClosedPublic

Authored by epriestley on Apr 28 2017, 12:22 AM.
Tags
None
Referenced Files
F13080589: D17800.diff
Wed, Apr 24, 9:06 AM
Unknown Object (File)
Fri, Apr 19, 6:26 PM
Unknown Object (File)
Wed, Apr 17, 2:34 PM
Unknown Object (File)
Wed, Apr 10, 7:42 PM
Unknown Object (File)
Fri, Apr 5, 12:58 PM
Unknown Object (File)
Fri, Apr 5, 11:25 AM
Unknown Object (File)
Sun, Mar 31, 9:37 AM
Unknown Object (File)
Jan 27 2024, 8:46 AM
Subscribers
None

Details

Summary

Ref T12651. Ran into these during D17799:

  • Use getStatusCode() to put the actual status code into the message.
  • If we fail but wrote an empty file to reserve the filename, clean it up.
Test Plan
  • Faked the error, phlog()'d the exception.
  • Saw sensible exception message.
  • Saw empty file get cleaned up.

Diff Detail

Repository
rARC Arcanist
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Apr 28 2017, 6:43 AM
This revision was automatically updated to reflect the committed changes.