Page MenuHomePhabricator

Allow "arc upload" to work correctly if it can not hash content
ClosedPublic

Authored by epriestley on Apr 4 2017, 10:48 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Jan 28, 2:26 AM
Unknown Object (File)
Fri, Jan 24, 8:02 PM
Unknown Object (File)
Fri, Jan 24, 8:02 PM
Unknown Object (File)
Fri, Jan 24, 8:02 PM
Unknown Object (File)
Fri, Jan 24, 8:02 PM
Unknown Object (File)
Mon, Jan 20, 9:25 PM
Unknown Object (File)
Sun, Jan 19, 10:12 AM
Unknown Object (File)
Mon, Jan 13, 6:43 PM
Subscribers
None

Details

Summary

Ref T12464. This is similar to D17619 and prepares us to move to SHA256 in the client.

Note that it's fine if arc and Phabricator disagree about hashing algorithms. We don't really trust the client anyway, so if things are mismatched clients will just end up transferring a bit more data instead of getting to cheat when Phabricator already has copies of data.

Test Plan

Ran arc upload, got a clean upload.

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 4 2017, 10:57 PM
This revision was automatically updated to reflect the committed changes.