HomePhabricator
Diffusion Arcanist a59cfca5f190

Upgrade "arc upload" to use SHA256
a59cfca5f190
Actions

Description

Upgrade "arc upload" to use SHA256

Summary: Fixes T12464. Moves "arc upload" to SHA256 where applicable.

Test Plan: Ran arc upload against a server with D17620 twice, saw it skip the actual upload the second time.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12464

Differential Revision: https://secure.phabricator.com/D17622

Details

Provenance
epriestleyAuthored on Apr 4 2017, 10:52 PM
epriestleyPushed on Apr 4 2017, 11:56 PM
Reviewer
chad
Differential Revision
D17622: Upgrade "arc upload" to use SHA256
Parents
rARCc4e84550fcd3: Allow "arc upload" to work correctly if it can not hash content
Branches
Unknown
Tags
Unknown
Tasks
T12464: Move away from SHA1 in Files
Build Status
Buildable 16312
Build 21686: Run Core Tests

Event Timeline

epriestley committed rARCa59cfca5f190: Upgrade "arc upload" to use SHA256 (authored by epriestley).Apr 4 2017, 11:56 PM
epriestley added a task: T12464: Move away from SHA1 in Files.
Harbormaster completed building B16312: rARCa59cfca5f190: Upgrade "arc upload" to use SHA256.Apr 4 2017, 11:57 PM
epriestley mentioned this in 2017 Week 14 (Early April).Apr 7 2017, 8:40 PM
joshuaspence mentioned this in T12531: Unable to upload file: failed to read 4583864320 bytes after offset 0.Apr 10 2017, 10:21 PM
joshuaspence mentioned this in T12534: Differential still appears in "responsible users" query after resigning as a reviewer.Apr 11 2017, 6:39 AM
fcoelho mentioned this in T12541: Unable to resolve method 'describeAutomaticCapability'.Apr 11 2017, 9:33 PM
QuietJoon mentioned this in T12543: `phd` can't handle PID properly when there are multiple phabricator daemons..Apr 12 2017, 6:20 AM
epriestley mentioned this in 2017 Week 15 (Mid April).Apr 14 2017, 8:37 PM
atul.agrawal9911 mentioned this in T12570: Installs without `gd` are unable to reach the setup warning telling them to install `gd`.Apr 17 2017, 1:09 PM
joshuaspence mentioned this in T12578: Herald rule for "Differential reviewers do not include" didn't fire when I expected it to.Apr 17 2017, 10:48 PM
atul.agrawal9911 mentioned this in T12579: ./bin/mail show-outbound command should type check <id>.Apr 18 2017, 5:38 AM
atul.agrawal9911 mentioned this in T12585: Unable to create differential .Apr 18 2017, 12:22 PM
atul.agrawal9911 mentioned this in T12729: CLI scripts do not detect execution with bad SAPI binary (like `fcgi` instead of `cli` under cPanel).May 19 2017, 9:52 AM

Changes (1)

PathSize
src/
upload/

rARCa59cfca5f190

View Options

src/upload/ArcanistFileDataRef.php

Loading...
Phabricator ยท Built by Phacility