data in PhabricatorFile hashed using sha1
Summary: Hash file content using sha1 Test Plan: None provided Reviewers: epriestley Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D4969
This commit is contained in:
@@ -121,7 +121,7 @@ final class PhabricatorFile extends PhabricatorFileDAO
|
|||||||
$file = id(new PhabricatorFile())->loadOneWhere(
|
$file = id(new PhabricatorFile())->loadOneWhere(
|
||||||
'name = %s AND contentHash = %s LIMIT 1',
|
'name = %s AND contentHash = %s LIMIT 1',
|
||||||
self::normalizeFileName(idx($params, 'name')),
|
self::normalizeFileName(idx($params, 'name')),
|
||||||
PhabricatorHash::digest($data));
|
self::hashFileContent($data));
|
||||||
|
|
||||||
if (!$file) {
|
if (!$file) {
|
||||||
$unguarded = AphrontWriteGuard::beginScopedUnguardedWrites();
|
$unguarded = AphrontWriteGuard::beginScopedUnguardedWrites();
|
||||||
@@ -231,7 +231,7 @@ final class PhabricatorFile extends PhabricatorFileDAO
|
|||||||
$file->setName($file_name);
|
$file->setName($file_name);
|
||||||
$file->setByteSize(strlen($data));
|
$file->setByteSize(strlen($data));
|
||||||
$file->setAuthorPHID($authorPHID);
|
$file->setAuthorPHID($authorPHID);
|
||||||
$file->setContentHash(PhabricatorHash::digest($data));
|
$file->setContentHash(self::hashFileContent($data));
|
||||||
|
|
||||||
$file->setStorageEngine($engine_identifier);
|
$file->setStorageEngine($engine_identifier);
|
||||||
$file->setStorageHandle($data_handle);
|
$file->setStorageHandle($data_handle);
|
||||||
@@ -373,7 +373,7 @@ final class PhabricatorFile extends PhabricatorFileDAO
|
|||||||
}
|
}
|
||||||
|
|
||||||
public static function hashFileContent($data) {
|
public static function hashFileContent($data) {
|
||||||
return PhabricatorHash::digest($data);
|
return sha1($data);
|
||||||
}
|
}
|
||||||
|
|
||||||
public function loadFileData() {
|
public function loadFileData() {
|
||||||
|
|||||||
Reference in New Issue
Block a user