Page MenuHomePhabricator

Fix two minor issues with "arc download"
ClosedPublic

Authored by epriestley on Apr 28 2017, 12:22 AM.
Tags
None
Referenced Files
F14388399: D17800.id42808.diff
Sat, Dec 21, 4:27 PM
F14388063: D17800.diff
Sat, Dec 21, 3:35 PM
Unknown Object (File)
Mon, Dec 9, 5:11 PM
Unknown Object (File)
Sat, Dec 7, 9:55 AM
Unknown Object (File)
Wed, Dec 4, 5:01 PM
Unknown Object (File)
Sat, Nov 30, 9:28 PM
Unknown Object (File)
Nov 19 2024, 1:18 PM
Unknown Object (File)
Oct 22 2024, 4:31 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.