spelling: names

[friendica.git] / src / Model / APContact.php

diff --git a/src/Model/APContact.php b/src/Model/APContact.php
index 71446e9696f1d100cbc78a624834fbc64246655f..dc062fe5b506afd222a92f7b4f8c75f70d413a0b 100644 (file)
--- a/src/Model/APContact.php
+++ b/src/Model/APContact.php
@@ -29,7 +29,6 @@ use Friendica\Core\System;
 use Friendica\Database\DBA;
 use Friendica\DI;
 use Friendica\Model\Item;
-use Friendica\Network\HTTPClient\Client\HttpClientAccept;
 use Friendica\Network\HTTPException;
 use Friendica\Network\Probe;
 use Friendica\Protocol\ActivityNamespace;
@@ -169,7 +168,7 @@ class APContact
                $cachekey = 'apcontact:' . ItemURI::getIdByURI($url);
                $result = DI::cache()->get($cachekey);
                if (!is_null($result)) {
-                       Logger::notice('Multiple requests for the address', ['url' => $url, 'update' => $update, 'callstack' => System::callstack(20), 'result' => $result]);
+                       Logger::info('Multiple requests for the address', ['url' => $url, 'update' => $update, 'callstack' => System::callstack(20), 'result' => $result]);
                        if (!empty($fetched_contact)) {
                                return $fetched_contact;
                        }
@@ -189,17 +188,22 @@ class APContact
                if (empty($data)) {
                        $local_owner = [];
 
-                       $curlResult = HTTPSignature::fetchRaw($url);
-                       $failed = empty($curlResult) || empty($curlResult->getBody()) ||
-                               (!$curlResult->isSuccess() && ($curlResult->getReturnCode() != 410));
-
-                       if (!$failed) {
-                               $data = json_decode($curlResult->getBody(), true);
-                               $failed = empty($data) || !is_array($data);
-                       }
+                       try {
+                               $curlResult = HTTPSignature::fetchRaw($url);
+                               $failed = empty($curlResult) || empty($curlResult->getBody()) ||
+                                       (!$curlResult->isSuccess() && ($curlResult->getReturnCode() != 410));
+       
+                               if (!$failed) {
+                                       $data = json_decode($curlResult->getBody(), true);
+                                       $failed = empty($data) || !is_array($data);
+                               }
 
-                       if (!$failed && ($curlResult->getReturnCode() == 410)) {
-                               $data = ['@context' => ActivityPub::CONTEXT, 'id' => $url, 'type' => 'Tombstone'];
+                               if (!$failed && ($curlResult->getReturnCode() == 410)) {
+                                       $data = ['@context' => ActivityPub::CONTEXT, 'id' => $url, 'type' => 'Tombstone'];
+                               }
+                       } catch (\Exception $exception) {
+                               Logger::notice('Error fetching url', ['url' => $url, 'exception' => $exception]);
+                               $failed = true;
                        }
 
                        if ($failed) {
@@ -254,6 +258,11 @@ class APContact
                $apcontact['photo'] = JsonLD::fetchElement($compacted, 'as:icon', '@id');
                if (is_array($apcontact['photo']) || !empty($compacted['as:icon']['as:url']['@id'])) {
                        $apcontact['photo'] = JsonLD::fetchElement($compacted['as:icon'], 'as:url', '@id');
+               } elseif (empty($apcontact['photo'])) {
+                       $photo = JsonLD::fetchElementArray($compacted, 'as:icon', 'as:url');
+                       if (!empty($photo[0]['@id'])) {
+                               $apcontact['photo'] = $photo[0]['@id'];
+                       }
                }
 
                $apcontact['header'] = JsonLD::fetchElement($compacted, 'as:image', '@id');