]> git.mxchange.org Git - friendica-addons.git/blobdiff - bluesky/bluesky.php
[bluesky] Fix double dollar sign in bluesky_fetch_notifications
[friendica-addons.git] / bluesky / bluesky.php
index cd1598dec901430d6644d77b29b68e40dbcd7251..c96658721f8737d020e7c38f8ff35897cf473b48 100644 (file)
@@ -9,6 +9,9 @@
  * Currently technical issues in the core:
  * - Outgoing mentions
  *
+ * At some point in time:
+ * - Sending Quote shares https://atproto.com/lexicons/app-bsky-embed#appbskyembedrecord and https://atproto.com/lexicons/app-bsky-embed#appbskyembedrecordwithmedia
+ *
  * Possibly not possible:
  * - only fetch new posts
  *
@@ -18,8 +21,6 @@
  *
  * Possibly interesting:
  * - https://atproto.com/lexicons/com-atproto-label#comatprotolabelsubscribelabels
- * - https://atproto.com/lexicons/com-atproto-repo#comatprotorepoputrecord
- * - https://atproto.com/lexicons/app-bsky-embed#appbskyembedrecordwithmedia
  */
 
 use Friendica\Content\Text\BBCode;
@@ -31,6 +32,7 @@ use Friendica\Core\Hook;
 use Friendica\Core\Logger;
 use Friendica\Core\Protocol;
 use Friendica\Core\Renderer;
+use Friendica\Core\System;
 use Friendica\Core\Worker;
 use Friendica\Database\DBA;
 use Friendica\DI;
@@ -41,6 +43,7 @@ use Friendica\Model\Photo;
 use Friendica\Model\Post;
 use Friendica\Network\HTTPClient\Client\HttpClientAccept;
 use Friendica\Network\HTTPClient\Client\HttpClientOptions;
+use Friendica\Object\Image;
 use Friendica\Protocol\Activity;
 use Friendica\Protocol\Relay;
 use Friendica\Util\DateTimeFormat;
@@ -48,6 +51,7 @@ use Friendica\Util\Strings;
 
 const BLUESKY_DEFAULT_POLL_INTERVAL = 10; // given in minutes
 const BLUESKY_HOST = 'https://bsky.app'; // Hard wired until Bluesky will run on multiple systems
+const BLUESKY_IMAGE_SIZE = [1000000, 500000, 100000, 50000];
 
 function bluesky_install()
 {
@@ -117,7 +121,7 @@ function bluesky_probe_detect(array &$hookData)
                return;
        }
 
-       $data = bluesky_get($pconfig['uid'], '/xrpc/app.bsky.actor.getProfile?actor=' . $did, HttpClientAccept::JSON, [HttpClientOptions::HEADERS => ['Authorization' => ['Bearer ' . $token]]]);
+       $data = bluesky_xrpc_get($pconfig['uid'], 'app.bsky.actor.getProfile', ['actor' => $did]);
        if (empty($data)) {
                return;
        }
@@ -162,7 +166,7 @@ function bluesky_item_by_link(array &$hookData)
 
        $uri = 'at://' . $did . '/app.bsky.feed.post/' . $matches[2];
 
-       $uri = bluesky_fetch_missing_post($uri, $hookData['uid'], 0);
+       $uri = bluesky_fetch_missing_post($uri, $hookData['uid'], 0, 0);
        Logger::debug('Got post', ['profile' => $matches[1], 'cid' => $matches[2], 'result' => $uri]);
        if (!empty($uri)) {
                $item = Post::selectFirst(['id'], ['uri' => $uri, 'uid' => $hookData['uid']]);
@@ -211,7 +215,7 @@ function bluesky_follow(array &$hook_data)
                'record'     => $record
        ];
 
-       $activity = bluesky_post($hook_data['uid'], '/xrpc/com.atproto.repo.createRecord', json_encode($post), ['Content-type' => 'application/json', 'Authorization' => ['Bearer ' . $token]]);
+       $activity = bluesky_xrpc_post($hook_data['uid'], 'com.atproto.repo.createRecord', $post);
        if (!empty($activity->uri)) {
                $hook_data['contact'] = $contact;
                Logger::debug('Successfully start following', ['url' => $contact['url'], 'uri' => $activity->uri]);
@@ -229,7 +233,7 @@ function bluesky_unfollow(array &$hook_data)
                return;
        }
 
-       $data = bluesky_get($hook_data['uid'], '/xrpc/app.bsky.actor.getProfile?actor=' . $hook_data['contact']['url'], HttpClientAccept::JSON, [HttpClientOptions::HEADERS => ['Authorization' => ['Bearer ' . $token]]]);
+       $data = bluesky_xrpc_get($hook_data['uid'], 'app.bsky.actor.getProfile', ['actor' => $hook_data['contact']['url']]);
        if (empty($data->viewer) || empty($data->viewer->following)) {
                return;
        }
@@ -264,7 +268,7 @@ function bluesky_block(array &$hook_data)
                'record'     => $record
        ];
 
-       $activity = bluesky_post($hook_data['uid'], '/xrpc/com.atproto.repo.createRecord', json_encode($post), ['Content-type' => 'application/json', 'Authorization' => ['Bearer ' . $token]]);
+       $activity = bluesky_xrpc_post($hook_data['uid'], 'com.atproto.repo.createRecord', $post);
        if (!empty($activity->uri)) {
                $cdata = Contact::getPublicAndUserContactID($hook_data['contact']['id'], $hook_data['uid']);
                if (!empty($cdata['user'])) {
@@ -285,7 +289,7 @@ function bluesky_unblock(array &$hook_data)
                return;
        }
 
-       $data = bluesky_get($hook_data['uid'], '/xrpc/app.bsky.actor.getProfile?actor=' . $hook_data['contact']['url'], HttpClientAccept::JSON, [HttpClientOptions::HEADERS => ['Authorization' => ['Bearer ' . $token]]]);
+       $data = bluesky_xrpc_get($hook_data['uid'], 'app.bsky.actor.getProfile', ['actor' => $hook_data['contact']['url']]);
        if (empty($data->viewer) || empty($data->viewer->blocking)) {
                return;
        }
@@ -419,22 +423,17 @@ function bluesky_cron()
                        }
                }
 
-               Logger::notice('importing timeline - start', ['user' => $pconfig['uid']]);
-               bluesky_fetch_timeline($pconfig['uid']);
-               Logger::notice('importing timeline - done', ['user' => $pconfig['uid']]);
+               // Refresh the token now, so that it doesn't need to be refreshed in parallel by the following workers
+               bluesky_get_token($pconfig['uid']);
 
-               Logger::notice('importing notifications - start', ['user' => $pconfig['uid']]);
-               bluesky_fetch_notifications($pconfig['uid']);
-               Logger::notice('importing notifications - done', ['user' => $pconfig['uid']]);
+               Worker::add(['priority' => Worker::PRIORITY_MEDIUM, 'force_priority' => true], 'addon/bluesky/bluesky_timeline.php', $pconfig['uid']);
+               Worker::add(['priority' => Worker::PRIORITY_MEDIUM, 'force_priority' => true], 'addon/bluesky/bluesky_notifications.php', $pconfig['uid']);
 
                if (DI::pConfig()->get($pconfig['uid'], 'bluesky', 'import_feeds')) {
-                       Logger::notice('importing feeds - start', ['user' => $pconfig['uid']]);
                        $feeds = bluesky_get_feeds($pconfig['uid']);
                        foreach ($feeds as $feed) {
-                               Logger::debug('Importing feed', ['user' => $pconfig['uid'], 'feed' => $feed]);
-                               bluesky_fetch_feed($pconfig['uid'], $feed);
+                               Worker::add(['priority' => Worker::PRIORITY_MEDIUM, 'force_priority' => true], 'addon/bluesky/bluesky_feed.php', $pconfig['uid'], $feed);
                        }
-                       Logger::notice('importing feeds - done', ['user' => $pconfig['uid']]);
                }
        }
 
@@ -594,7 +593,7 @@ function bluesky_create_activity(array $item, stdClass $parent = null)
                ];
        }
 
-       $activity = bluesky_post($uid, '/xrpc/com.atproto.repo.createRecord', json_encode($post), ['Content-type' => 'application/json', 'Authorization' => ['Bearer ' . $token]]);
+       $activity = bluesky_xrpc_post($uid, 'com.atproto.repo.createRecord', $post);
        if (empty($activity)) {
                return;
        }
@@ -637,6 +636,12 @@ function bluesky_create_post(array $item, stdClass $root = null, stdClass $paren
 
                if ($key == count($msg['parts']) - 1) {
                        $record = bluesky_add_embed($uid, $msg, $record);
+                       if (empty($record)) {
+                               if (Worker::getRetrial() < 3) {
+                                       Worker::defer();
+                               }
+                               return;
+                       }
                }
 
                $post = [
@@ -645,8 +650,11 @@ function bluesky_create_post(array $item, stdClass $root = null, stdClass $paren
                        'record'     => $record
                ];
 
-               $parent = bluesky_post($uid, '/xrpc/com.atproto.repo.createRecord', json_encode($post), ['Content-type' => 'application/json', 'Authorization' => ['Bearer ' . $token]]);
+               $parent = bluesky_xrpc_post($uid, 'com.atproto.repo.createRecord', $post);
                if (empty($parent)) {
+                       if ($part == 0) {
+                               Worker::defer();
+                       }
                        return;
                }
                Logger::debug('Posting done', ['return' => $parent]);
@@ -663,9 +671,10 @@ function bluesky_create_post(array $item, stdClass $root = null, stdClass $paren
 
 function bluesky_get_urls(string $body): array
 {
-       // Remove all hashtags and mentions
-       $body = preg_replace("/([#@!])\[url\=(.*?)\](.*?)\[\/url\]/ism", '', $body);
+       // Remove all hashtag and mention links
+       $body = preg_replace("/([#@!])\[url\=(.*?)\](.*?)\[\/url\]/ism", '$1$3', $body);
 
+       $body = BBCode::expandVideoLinks($body);
        $urls = [];
 
        // Search for pure links
@@ -749,12 +758,15 @@ function bluesky_add_embed(int $uid, array $msg, array $record): array
        if (($msg['type'] != 'link') && !empty($msg['images'])) {
                $images = [];
                foreach ($msg['images'] as $image) {
-                       $photo = Photo::selectFirst(['resource-id'], ['id' => $image['id']]);
-                       $photo = Photo::selectFirst([], ["`resource-id` = ? AND `scale` > ?", $photo['resource-id'], 0], ['order' => ['scale']]);
+                       if (count($images) == 4) {
+                               continue;
+                       }
+                       $photo = Photo::selectFirst([], ['id' => $image['id']]);
                        $blob = bluesky_upload_blob($uid, $photo);
-                       if (!empty($blob) && count($images) < 4) {
-                               $images[] = ['alt' => $image['description'] ?? '', 'image' => $blob];
+                       if (empty($blob)) {
+                               return [];
                        }
+                       $images[] = ['alt' => $image['description'] ?? '', 'image' => $blob];
                }
                if (!empty($images)) {
                        $record['embed'] = ['$type' => 'app.bsky.embed.images', 'images' => $images];
@@ -764,8 +776,8 @@ function bluesky_add_embed(int $uid, array $msg, array $record): array
                        '$type'    => 'app.bsky.embed.external',
                        'external' => [
                                'uri'         => $msg['url'],
-                               'title'       => $msg['title'],
-                               'description' => $msg['description'],
+                               'title'       => $msg['title'] ?? '',
+                               'description' => $msg['description'] ?? '',
                        ]
                ];
                if (!empty($msg['image'])) {
@@ -781,31 +793,46 @@ function bluesky_add_embed(int $uid, array $msg, array $record): array
 
 function bluesky_upload_blob(int $uid, array $photo): ?stdClass
 {
+       $retrial = Worker::getRetrial();
        $content = Photo::getImageForPhoto($photo);
+
+       $picture = new Image($content, $photo['type']);
+       $height  = $picture->getHeight();
+       $width   = $picture->getWidth();
+       $size    = strlen($content);
+
+       $picture    = Photo::resizeToFileSize($picture, BLUESKY_IMAGE_SIZE[$retrial]);
+       $new_height = $picture->getHeight();
+       $new_width  = $picture->getWidth();
+       $content    = $picture->asString();
+       $new_size   = strlen($content);
+
+       Logger::info('Uploading', ['uid' => $uid, 'retrial' => $retrial, 'height' => $new_height, 'width' => $new_width, 'size' => $new_size, 'orig-height' => $height, 'orig-width' => $width, 'orig-size' => $size]);
+
        $data = bluesky_post($uid, '/xrpc/com.atproto.repo.uploadBlob', $content, ['Content-type' => $photo['type'], 'Authorization' => ['Bearer ' . bluesky_get_token($uid)]]);
        if (empty($data)) {
+               Logger::info('Uploading failed', ['uid' => $uid, 'retrial' => $retrial, 'height' => $new_height, 'width' => $new_width, 'size' => $new_size, 'orig-height' => $height, 'orig-width' => $width, 'orig-size' => $size]);
                return null;
        }
 
-       Logger::debug('Uploaded blob', ['return' => $data]);
+       Logger::debug('Uploaded blob', ['return' => $data, 'uid' => $uid, 'retrial' => $retrial, 'height' => $new_height, 'width' => $new_width, 'size' => $new_size, 'orig-height' => $height, 'orig-width' => $width, 'orig-size' => $size]);
        return $data->blob;
 }
 
 function bluesky_delete_post(string $uri, int $uid)
 {
-       $token = bluesky_get_token($uid);
        $parts = bluesky_get_uri_parts($uri);
        if (empty($parts)) {
                Logger::debug('No uri delected', ['uri' => $uri]);
                return;
        }
-       bluesky_post($uid, '/xrpc/com.atproto.repo.deleteRecord', json_encode($parts), ['Content-type' => 'application/json', 'Authorization' => ['Bearer ' . $token]]);
+       bluesky_xrpc_post($uid, 'com.atproto.repo.deleteRecord', $parts);
        Logger::debug('Deleted', ['parts' => $parts]);
 }
 
 function bluesky_fetch_timeline(int $uid)
 {
-       $data = bluesky_get($uid, '/xrpc/app.bsky.feed.getTimeline', HttpClientAccept::JSON, [HttpClientOptions::HEADERS => ['Authorization' => ['Bearer ' . bluesky_get_token($uid)]]]);
+       $data = bluesky_xrpc_get($uid, 'app.bsky.feed.getTimeline');
        if (empty($data)) {
                return;
        }
@@ -815,7 +842,7 @@ function bluesky_fetch_timeline(int $uid)
        }
 
        foreach (array_reverse($data->feed) as $entry) {
-               bluesky_process_post($entry->post, $uid, Item::PR_NONE);
+               bluesky_process_post($entry->post, $uid, Item::PR_NONE, 0);
                if (!empty($entry->reason)) {
                        bluesky_process_reason($entry->reason, bluesky_get_uri($entry->post), $uid);
                }
@@ -856,6 +883,7 @@ function bluesky_process_reason(stdClass $reason, string $uri, int $uid)
                return;
        }
 
+       $item['guid']         = Item::guidFromUri($item['uri'], $contact['alias']);
        $item['owner-name']   = $item['author-name'];
        $item['owner-link']   = $item['author-link'];
        $item['owner-avatar'] = $item['author-avatar'];
@@ -867,11 +895,11 @@ function bluesky_process_reason(stdClass $reason, string $uri, int $uid)
 
 function bluesky_fetch_notifications(int $uid)
 {
-       $result = bluesky_get($uid, '/xrpc/app.bsky.notification.listNotifications', HttpClientAccept::JSON, [HttpClientOptions::HEADERS => ['Authorization' => ['Bearer ' . bluesky_get_token($uid)]]]);
-       if (empty($result->notifications)) {
+       $data = bluesky_xrpc_get($uid, 'app.bsky.notification.listNotifications');
+       if (empty($data->notifications)) {
                return;
        }
-       foreach ($result->notifications as $notification) {
+       foreach ($data->notifications as $notification) {
                $uri = bluesky_get_uri($notification);
                if (Post::exists(['uri' => $uri, 'uid' => $uid]) || Post::exists(['extid' => $uri, 'uid' => $uid])) {
                        Logger::debug('Notification already processed', ['uid' => $uid, 'reason' => $notification->reason, 'uri' => $uri, 'indexedAt' => $notification->indexedAt]);
@@ -884,20 +912,28 @@ function bluesky_fetch_notifications(int $uid)
                                $item['gravity'] = Item::GRAVITY_ACTIVITY;
                                $item['body'] = $item['verb'] = Activity::LIKE;
                                $item['thr-parent'] = bluesky_get_uri($notification->record->subject);
-                               $item['thr-parent'] = bluesky_fetch_missing_post($item['thr-parent'], $uid, $item['contact-id']);
-                               $result = Item::insert($item);
-                               Logger::debug('Got like', ['uid' => $uid, 'result' => $result, 'uri' => $uri]);
-                               break;
+                               $item['thr-parent'] = bluesky_fetch_missing_post($item['thr-parent'], $uid, $item['contact-id'], 0);
+                               if (!empty($item['thr-parent'])) {
+                                       $data = Item::insert($item);
+                                       Logger::debug('Got like', ['uid' => $uid, 'result' => $data, 'uri' => $uri]);
+                               } else {
+                                       Logger::info('Thread parent not found', ['uid' => $uid, 'parent' => $item['thr-parent'], 'uri' => $uri]);
+                               }
+                       break;
 
                        case 'repost':
                                $item = bluesky_get_header($notification, $uri, $uid, $uid);
                                $item['gravity'] = Item::GRAVITY_ACTIVITY;
                                $item['body'] = $item['verb'] = Activity::ANNOUNCE;
                                $item['thr-parent'] = bluesky_get_uri($notification->record->subject);
-                               $item['thr-parent'] = bluesky_fetch_missing_post($item['thr-parent'], $uid, $item['contact-id']);
-                               $result = Item::insert($item);
-                               Logger::debug('Got repost', ['uid' => $uid, 'result' => $result, 'uri' => $uri]);
-                               break;
+                               $item['thr-parent'] = bluesky_fetch_missing_post($item['thr-parent'], $uid, $item['contact-id'], 0);
+                               if (!empty($item['thr-parent'])) {
+                                       $data = Item::insert($item);
+                                       Logger::debug('Got repost', ['uid' => $uid, 'result' => $data, 'uri' => $uri]);
+                               } else {
+                                       Logger::info('Thread parent not found', ['uid' => $uid, 'parent' => $$item['thr-parent'], 'uri' => $uri]);
+                               }
+                       break;
 
                        case 'follow':
                                $contact = bluesky_get_contact($notification->author, $uid, $uid);
@@ -905,18 +941,18 @@ function bluesky_fetch_notifications(int $uid)
                                break;
 
                        case 'mention':
-                               $result = bluesky_process_post($notification, $uid, Item::PR_PUSHED);
-                               Logger::debug('Got mention', ['uid' => $uid, 'result' => $result, 'uri' => $uri]);
+                               $data = bluesky_process_post($notification, $uid, Item::PR_PUSHED, 0);
+                               Logger::debug('Got mention', ['uid' => $uid, 'result' => $data, 'uri' => $uri]);
                                break;
 
                        case 'reply':
-                               $result = bluesky_process_post($notification, $uid, Item::PR_PUSHED);
-                               Logger::debug('Got reply', ['uid' => $uid, 'result' => $result, 'uri' => $uri]);
+                               $data = bluesky_process_post($notification, $uid, Item::PR_PUSHED, 0);
+                               Logger::debug('Got reply', ['uid' => $uid, 'result' => $data, 'uri' => $uri]);
                                break;
 
                        case 'quote':
-                               $result = bluesky_process_post($notification, $uid, Item::PR_PUSHED);
-                               Logger::debug('Got quote', ['uid' => $uid, 'result' => $result, 'uri' => $uri]);
+                               $data = bluesky_process_post($notification, $uid, Item::PR_PUSHED, 0);
+                               Logger::debug('Got quote', ['uid' => $uid, 'result' => $data, 'uri' => $uri]);
                                break;
 
                        default:
@@ -928,7 +964,7 @@ function bluesky_fetch_notifications(int $uid)
 
 function bluesky_fetch_feed(int $uid, string $feed)
 {
-       $data = bluesky_get($uid, '/xrpc/app.bsky.feed.getFeed?feed=' . $feed, HttpClientAccept::JSON, [HttpClientOptions::HEADERS => ['Authorization' => ['Bearer ' . bluesky_get_token($uid)]]]);
+       $data = bluesky_xrpc_get($uid, 'app.bsky.feed.getFeed', ['feed' => $feed]);
        if (empty($data)) {
                return;
        }
@@ -942,14 +978,14 @@ function bluesky_fetch_feed(int $uid, string $feed)
                        Logger::debug('Unwanted language detected', ['text' => $entry->post->record->text]);
                        continue;
                }
-               bluesky_process_post($entry->post, $uid, Item::PR_TAG);
+               bluesky_process_post($entry->post, $uid, Item::PR_TAG, 0);
                if (!empty($entry->reason)) {
                        bluesky_process_reason($entry->reason, bluesky_get_uri($entry->post), $uid);
                }
        }
 }
 
-function bluesky_process_post(stdClass $post, int $uid, int $post_reason): int
+function bluesky_process_post(stdClass $post, int $uid, int $post_reason, $level): int
 {
        $uri = bluesky_get_uri($post);
 
@@ -957,14 +993,16 @@ function bluesky_process_post(stdClass $post, int $uid, int $post_reason): int
                return 0;
        }
 
-       Logger::debug('Importing post', ['uid' => $uid, 'indexedAt' => $post->indexedAt, 'uri' => $post->uri, 'cid' => $post->cid]);
+       Logger::debug('Importing post', ['uid' => $uid, 'indexedAt' => $post->indexedAt, 'uri' => $post->uri, 'cid' => $post->cid, 'root' => $post->record->reply->root ?? '']);
 
        $item = bluesky_get_header($post, $uri, $uid, $uid);
-
-       $item = bluesky_get_content($item, $post->record, $uid);
+       $item = bluesky_get_content($item, $post->record, $uri, $uid, $level);
+       if (empty($item)) {
+               return 0;
+       }
 
        if (!empty($post->embed)) {
-               $item = bluesky_add_media($post->embed, $item, $uid);
+               $item = bluesky_add_media($post->embed, $item, $uid, $level);
        }
 
        if (empty($item['post-reason'])) {
@@ -1008,21 +1046,36 @@ function bluesky_get_header(stdClass $post, string $uri, int $uid, int $fetch_ui
        return $item;
 }
 
-function bluesky_get_content(array $item, stdClass $record, int $uid): array
+function bluesky_get_content(array $item, stdClass $record, string $uri, int $uid, int $level): array
 {
+       if (empty($item)) {
+               return [];
+       }
+
        if (!empty($record->reply)) {
                $item['parent-uri'] = bluesky_get_uri($record->reply->root);
-               $item['parent-uri'] = bluesky_fetch_missing_post($item['parent-uri'], $uid, $item['contact-id']);
+               if ($item['parent-uri'] != $uri) {
+                       $item['parent-uri'] = bluesky_fetch_missing_post($item['parent-uri'], $uid, $item['contact-id'], $level);
+                       if (empty($item['parent-uri'])) {
+                               return [];
+                       }
+               }
+
                $item['thr-parent'] = bluesky_get_uri($record->reply->parent);
-               $item['thr-parent'] = bluesky_fetch_missing_post($item['thr-parent'], $uid, $item['contact-id']);
+               if (!in_array($item['thr-parent'], [$uri, $item['parent-uri']])) {
+                       $item['thr-parent'] = bluesky_fetch_missing_post($item['thr-parent'], $uid, $item['contact-id'], $level, $item['parent-uri']);
+                       if (empty($item['thr-parent'])) {
+                               return [];
+                       }
+               }
        }
 
-       $item['body']    = bluesky_get_text($record, $uid);
+       $item['body']    = bluesky_get_text($record);
        $item['created'] = DateTimeFormat::utc($record->createdAt, DateTimeFormat::MYSQL);
        return $item;
 }
 
-function bluesky_get_text(stdClass $record, int $uid): string
+function bluesky_get_text(stdClass $record): string
 {
        $text = $record->text;
 
@@ -1073,7 +1126,7 @@ function bluesky_get_text(stdClass $record, int $uid): string
        return $text;
 }
 
-function bluesky_add_media(stdClass $embed, array $item, int $fetch_uid): array
+function bluesky_add_media(stdClass $embed, array $item, int $fetch_uid, int $level): array
 {
        $type = '$type';
        switch ($embed->$type) {
@@ -1105,13 +1158,16 @@ function bluesky_add_media(stdClass $embed, array $item, int $fetch_uid): array
                        $uri = bluesky_get_uri($embed->record);
                        $shared = Post::selectFirst(['uri-id'], ['uri' => $uri, 'uid' => $item['uid']]);
                        if (empty($shared)) {
+                               if (empty($embed->record->value)) {
+                                       Logger::info('Record has got no value', ['record' => $embed->record]);
+                                       break;
+                               }
                                $shared = bluesky_get_header($embed->record, $uri, 0, $fetch_uid);
+                               $shared = bluesky_get_content($shared, $embed->record->value, $uri, $item['uid'], $level);
                                if (!empty($shared)) {
-                                       $shared = bluesky_get_content($shared, $embed->record->value, $item['uid']);
-
                                        if (!empty($embed->record->embeds)) {
                                                foreach ($embed->record->embeds as $single) {
-                                                       $shared = bluesky_add_media($single, $shared, $fetch_uid);
+                                                       $shared = bluesky_add_media($single, $shared, $fetch_uid, $level);
                                                }
                                        }
                                        $id = Item::insert($shared);
@@ -1128,19 +1184,14 @@ function bluesky_add_media(stdClass $embed, array $item, int $fetch_uid): array
                        $shared = Post::selectFirst(['uri-id'], ['uri' => $uri, 'uid' => $item['uid']]);
                        if (empty($shared)) {
                                $shared = bluesky_get_header($embed->record->record, $uri, 0, $fetch_uid);
+                               $shared = bluesky_get_content($shared, $embed->record->record->value, $uri, $item['uid'], $level);
                                if (!empty($shared)) {
-                                       $shared = bluesky_get_content($shared, $embed->record->record->value, $item['uid']);
-
-                                       if (!empty($embed->record->embeds)) {
+                                       if (!empty($embed->record->record->embeds)) {
                                                foreach ($embed->record->record->embeds as $single) {
-                                                       $shared = bluesky_add_media($single, $shared, $fetch_uid);
+                                                       $shared = bluesky_add_media($single, $shared, $fetch_uid, $level);
                                                }
                                        }
 
-                                       if (!empty($embed->media)) {
-                                               bluesky_add_media($embed->media, $item, $fetch_uid);
-                                       }
-
                                        $id = Item::insert($shared);
                                        $shared = Post::selectFirst(['uri-id'], ['id' => $id]);
                                }
@@ -1148,6 +1199,10 @@ function bluesky_add_media(stdClass $embed, array $item, int $fetch_uid): array
                        if (!empty($shared)) {
                                $item['quote-uri-id'] = $shared['uri-id'];
                        }
+
+                       if (!empty($embed->media)) {
+                               bluesky_add_media($embed->media, $item, $fetch_uid, $level);
+                       }
                        break;
 
                default:
@@ -1159,6 +1214,10 @@ function bluesky_add_media(stdClass $embed, array $item, int $fetch_uid): array
 
 function bluesky_get_uri(stdClass $post): string
 {
+       if (empty($post->cid)) {
+               Logger::info('Invalid URI', ['post' => $post, 'callstack' => System::callstack(10, 0, true)]);
+               return '';
+       }
        return $post->uri . ':' . $post->cid;
 }
 
@@ -1205,27 +1264,39 @@ function bluesky_get_uri_parts(string $uri): ?stdClass
        return $class;
 }
 
-function bluesky_fetch_missing_post(string $uri, int $uid, int $causer): string
+function bluesky_fetch_missing_post(string $uri, int $uid, int $causer, int $level, string $fallback = ''): string
 {
        $fetched_uri = bluesky_fetch_post($uri, $uid);
        if (!empty($fetched_uri)) {
                return $fetched_uri;
        }
 
-       Logger::debug('Fetch missing post', ['uri' => $uri]);
+       if (++$level > 100) {
+               Logger::info('Recursion level too deep', ['level' => $level, 'uid' => $uid, 'uri' => $uri, 'fallback' => $fallback]);
+               // When the level is too deep we will fallback to the parent uri.
+               // Allthough the threading won't be correct, we at least had stored all posts and won't try again
+               return $fallback;
+       }
+
        $class = bluesky_get_uri_class($uri);
        $fetch_uri = $class->uri;
 
-       $data = bluesky_get($uid, '/xrpc/app.bsky.feed.getPostThread?uri=' . urlencode($fetch_uri), HttpClientAccept::JSON, [HttpClientOptions::HEADERS => ['Authorization' => ['Bearer ' . bluesky_get_token($uid)]]]);
+       Logger::debug('Fetch missing post', ['level' => $level, 'uid' => $uid, 'uri' => $uri]);
+       $data = bluesky_xrpc_get($uid, 'app.bsky.feed.getPostThread', ['uri' => $fetch_uri]);
        if (empty($data)) {
-               return '';
+               Logger::info('Thread was not fetched', ['level' => $level, 'uid' => $uid, 'uri' => $uri, 'fallback' => $fallback]);
+               return $fallback;
        }
 
+       Logger::debug('Reply count', ['replies' => $data->thread->post->replyCount, 'level' => $level, 'uid' => $uid, 'uri' => $uri]);
+
        if ($causer != 0) {
                $cdata = Contact::getPublicAndUserContactID($causer, $uid);
+       } else {
+               $cdata = [];
        }
 
-       return bluesky_process_thread($data->thread, $uid, $cdata);
+       return bluesky_process_thread($data->thread, $uid, $cdata, $level);
 }
 
 function bluesky_fetch_post(string $uri, int $uid): string
@@ -1243,33 +1314,42 @@ function bluesky_fetch_post(string $uri, int $uid): string
        return '';
 }
 
-function bluesky_process_thread(stdClass $thread, int $uid, array $cdata): string
+function bluesky_process_thread(stdClass $thread, int $uid, array $cdata, int $level): string
 {
        $uri = bluesky_get_uri($thread->post);
        $fetched_uri = bluesky_fetch_post($uri, $uid);
        if (empty($fetched_uri)) {
                Logger::debug('Process missing post', ['uri' => $uri]);
                $item = bluesky_get_header($thread->post, $uri, $uid, $uid);
-               $item = bluesky_get_content($item, $thread->post->record, $uid);
+               $item = bluesky_get_content($item, $thread->post->record, $uri, $uid, $level);
+               if (!empty($item)) {
+                       $item['post-reason'] = Item::PR_FETCHED;
 
-               $item['post-reason'] = Item::PR_FETCHED;
-
-               if (!empty($cdata['public'])) {
-                       $item['causer-id']   = $cdata['public'];
-               }
+                       if (!empty($cdata['public'])) {
+                               $item['causer-id']   = $cdata['public'];
+                       }
 
-               if (!empty($thread->post->embed)) {
-                       $item = bluesky_add_media($thread->post->embed, $item, $uid);
+                       if (!empty($thread->post->embed)) {
+                               $item = bluesky_add_media($thread->post->embed, $item, $uid, $level);
+                       }
+                       $id = Item::insert($item);
+                       if (!$id) {
+                               Logger::info('Item has not not been stored', ['uri' => $uri]);
+                               return '';
+                       }
+                       Logger::debug('Stored item', ['id' => $id, 'uri' => $uri]);
+               } else {
+                       Logger::info('Post has not not been fetched', ['uri' => $uri]);
+                       return '';
                }
-               $id = Item::insert($item);
-               Logger::debug('Stored item', ['id' => $id, 'uri' => $uri]);
        } else {
                Logger::debug('Post exists', ['uri' => $uri]);
                $uri = $fetched_uri;
        }
 
-       foreach ($thread->replies as $reply) {
-               bluesky_process_thread($reply, $uid, $cdata);
+       foreach ($thread->replies ?? [] as $reply) {
+               $reply_uri = bluesky_process_thread($reply, $uid, $cdata, $level);
+               Logger::debug('Reply has been processed', ['uri' => $uri, 'reply' => $reply_uri]);
        }
 
        return $uri;
@@ -1347,7 +1427,7 @@ function bluesky_get_contact_fields(stdClass $author, int $uid, bool $update): a
                return $fields;
        }
 
-       $data = bluesky_get($uid, '/xrpc/app.bsky.actor.getProfile?actor=' . $author->did, HttpClientAccept::JSON, [HttpClientOptions::HEADERS => ['Authorization' => ['Bearer ' . bluesky_get_token($uid)]]]);
+       $data = bluesky_xrpc_get($uid, 'app.bsky.actor.getProfile', ['actor' => $author->did]);
        if (empty($data)) {
                Logger::debug('Error fetching contact fields', ['uid' => $uid, 'url' => $fields['url']]);
                return $fields;
@@ -1399,7 +1479,7 @@ function bluesky_get_preferences(int $uid): stdClass
                return $data;
        }
 
-       $data = bluesky_get($uid, '/xrpc/app.bsky.actor.getPreferences', HttpClientAccept::JSON, [HttpClientOptions::HEADERS => ['Authorization' => ['Bearer ' . bluesky_get_token($uid)]]]);
+       $data = bluesky_xrpc_get($uid, 'app.bsky.actor.getPreferences');
 
        DI::cache()->set($cachekey, $data, Duration::HOUR);
        return $data;
@@ -1407,7 +1487,7 @@ function bluesky_get_preferences(int $uid): stdClass
 
 function bluesky_get_did(int $uid, string $handle): string
 {
-       $data = bluesky_get($uid, '/xrpc/com.atproto.identity.resolveHandle?handle=' . $handle);
+       $data = bluesky_get($uid, '/xrpc/com.atproto.identity.resolveHandle?handle=' . urlencode($handle));
        if (empty($data)) {
                return '';
        }
@@ -1461,6 +1541,11 @@ function bluesky_create_token(int $uid, string $password): string
        return $data->accessJwt;
 }
 
+function bluesky_xrpc_post(int $uid, string $url, $parameters): ?stdClass
+{
+       return bluesky_post($uid, '/xrpc/' . $url, json_encode($parameters),  ['Content-type' => 'application/json', 'Authorization' => ['Bearer ' . bluesky_get_token($uid)]]);
+}
+
 function bluesky_post(int $uid, string $url, string $params, array $headers): ?stdClass
 {
        try {
@@ -1478,6 +1563,15 @@ function bluesky_post(int $uid, string $url, string $params, array $headers): ?s
        return json_decode($curlResult->getBody());
 }
 
+function bluesky_xrpc_get(int $uid, string $url, array $parameters = []): ?stdClass
+{
+       if (!empty($parameters)) {
+               $url .= '?' . http_build_query($parameters);
+       }
+
+       return bluesky_get($uid, '/xrpc/' . $url, HttpClientAccept::JSON, [HttpClientOptions::HEADERS => ['Authorization' => ['Bearer ' . bluesky_get_token($uid)]]]);
+}
+
 function bluesky_get(int $uid, string $url, string $accept_content = HttpClientAccept::DEFAULT, array $opts = []): ?stdClass
 {
        try {
@@ -1493,4 +1587,4 @@ function bluesky_get(int $uid, string $url, string $accept_content = HttpClientA
        }
 
        return json_decode($curlResult->getBody());
-}
\ No newline at end of file
+}