Page MenuHomePhabricator

Fix two minor issues with "arc download"
ClosedPublic

Authored by epriestley on Apr 28 2017, 12:22 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Aug 16, 1:16 PM
Unknown Object (File)
Thu, Jul 30, 1:26 AM
Unknown Object (File)
Jul 10 2026, 11:48 AM
Unknown Object (File)
Apr 12 2026, 7:37 PM
Unknown Object (File)
Mar 28 2026, 4:38 AM
Unknown Object (File)
Mar 27 2026, 8:57 PM
Unknown Object (File)
Feb 9 2026, 7:19 PM
Unknown Object (File)
Feb 4 2026, 4:49 PM
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.