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
F18791407: D17621.id.diff
Thu, Oct 16, 12:38 PM
F18789115: D17621.id42373.diff
Wed, Oct 15, 8:05 AM
F18762440: D17621.id.diff
Mon, Oct 6, 7:02 PM
F18758185: D17621.diff
Sun, Oct 5, 9:00 PM
F18700673: D17621.diff
Sat, Sep 27, 3:37 PM
F18684409: D17621.diff
Fri, Sep 26, 9:28 AM
F18684393: D17621.diff
Fri, Sep 26, 9:22 AM
F18684376: D17621.diff
Fri, Sep 26, 9:17 AM
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.