Page MenuHomePhabricator

Deprecate "file.uploadhash"
ClosedPublic

Authored by epriestley on Apr 4 2017, 9:40 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 10, 2:01 AM
Unknown Object (File)
Tue, Apr 9, 11:33 PM
Unknown Object (File)
Tue, Apr 9, 5:55 PM
Unknown Object (File)
Tue, Apr 9, 5:55 PM
Unknown Object (File)
Tue, Apr 9, 5:22 PM
Unknown Object (File)
Tue, Apr 9, 12:35 PM
Unknown Object (File)
Mon, Apr 8, 10:41 PM
Unknown Object (File)
Sat, Apr 6, 6:05 AM
Subscribers
None

Details

Summary

Ref T12464. This is a very old method which let you create a file on the server by referring to data which already existed in another file.

Basically, long ago, arc could say "Do you already have a file with hash X?" and just skip some work if the server did.

arc has not called this method since D13017, in May 2015.

Since it's easy to do so, just make this method pretend that it never has the file. Very old clients will continue to work, since they would expect this response in the common case and continue by uploading data.

Test Plan
  • Grepped for uploadhash in Phabricator and Arcanist.
  • Called the method with the console, verified it returned null.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.Apr 4 2017, 10:30 PM
This revision was automatically updated to reflect the committed changes.