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)
Mon, Feb 27, 1:58 PM
Unknown Object (File)
Feb 19 2023, 8:23 PM
Unknown Object (File)
Dec 17 2022, 2:59 AM
Unknown Object (File)
Nov 13 2022, 11:19 AM
Unknown Object (File)
Nov 12 2022, 11:08 PM
Unknown Object (File)
Oct 30 2022, 7:18 PM
Unknown Object (File)
Oct 30 2022, 1:31 AM
Unknown Object (File)
Oct 22 2022, 1:02 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.