Avoid unmarking for archival an empty contact in Protocol\ActivityPub\Receiver

- Address https://github.com/friendica/friendica/issues/9250#issuecomment-733198306
This commit is contained in:
Hypolite Petovan 2020-11-24 17:32:52 -05:00
parent 1342825401
commit 486b43f09d

View File

@ -99,13 +99,15 @@ class Receiver
$actor = JsonLD::fetchElement($ldactivity, 'as:actor', '@id'); $actor = JsonLD::fetchElement($ldactivity, 'as:actor', '@id');
$apcontact = APContact::getByURL($actor); $apcontact = APContact::getByURL($actor);
if (!empty($apcontact) && ($apcontact['type'] == 'Application') && ($apcontact['nick'] == 'relay')) { if (empty($apcontact)) {
Logger::notice('Unable to retrieve AP contact for actor', ['actor' => $actor]);
} elseif ($apcontact['type'] == 'Application' && $apcontact['nick'] == 'relay') {
self::processRelayPost($ldactivity, $actor); self::processRelayPost($ldactivity, $actor);
return; return;
} else {
APContact::unmarkForArchival($apcontact);
} }
APContact::unMarkForArchival($apcontact);
$http_signer = HTTPSignature::getSigner($body, $header); $http_signer = HTTPSignature::getSigner($body, $header);
if (empty($http_signer)) { if (empty($http_signer)) {
Logger::warning('Invalid HTTP signature, message will be discarded.'); Logger::warning('Invalid HTTP signature, message will be discarded.');
@ -235,7 +237,7 @@ class Receiver
$profile = APContact::getByURL($object_id); $profile = APContact::getByURL($object_id);
if (!empty($profile['type'])) { if (!empty($profile['type'])) {
APContact::unMarkForArchival($profile); APContact::unmarkForArchival($profile);
return 'as:' . $profile['type']; return 'as:' . $profile['type'];
} }