Fix API issues #13887, #13886, #13863, #13809, #13897 (#13911)
authorMichael Vogel <icarus@dabo.de>
Sun, 18 Feb 2024 14:52:30 +0000 (15:52 +0100)
committerGitHub <noreply@github.com>
Sun, 18 Feb 2024 14:52:30 +0000 (15:52 +0100)
src/Factory/Api/Mastodon/Application.php
src/Module/Api/Mastodon/Apps/VerifyCredentials.php
src/Module/Api/Mastodon/Instance.php
src/Module/Api/Mastodon/Instance/Extended.php [new file with mode: 0644]
src/Module/Api/Mastodon/Instance/Peers.php
src/Module/Api/Mastodon/Instance/Rules.php
src/Module/Api/Mastodon/Search.php
src/Object/Api/Mastodon/Application.php
src/Object/Api/Mastodon/Card.php
src/Object/Api/Mastodon/ExtendedDescription.php [new file with mode: 0644]
static/routes.config.php

index da605ea..6062b88 100644 (file)
@@ -23,6 +23,7 @@ namespace Friendica\Factory\Api\Mastodon;
 
 use Friendica\BaseFactory;
 use Friendica\Database\Database;
+use Friendica\Model\Subscription;
 use Friendica\Network\HTTPException\UnprocessableEntityException;
 use Psr\Log\LoggerInterface;
 
@@ -56,6 +57,8 @@ class Application extends BaseFactory
                        $application['client_secret'],
                        $application['id'],
                        $application['redirect_uri'],
-                       $application['website']);
+                       $application['website'],
+                       Subscription::getPublicVapidKey(),
+               );
        }
 }
index 6b5f49c..961f5f3 100644 (file)
@@ -21,7 +21,6 @@
 
 namespace Friendica\Module\Api\Mastodon\Apps;
 
-use Friendica\Core\System;
 use Friendica\DI;
 use Friendica\Module\BaseApi;
 
index 8faa356..63d77f2 100644 (file)
@@ -38,7 +38,7 @@ use Friendica\Util\Strings;
 use Psr\Log\LoggerInterface;
 
 /**
- * @see https://docs.joinmastodon.org/api/rest/instances/
+ * @see https://docs.joinmastodon.org/methods/instance/
  */
 class Instance extends BaseApi
 {
diff --git a/src/Module/Api/Mastodon/Instance/Extended.php b/src/Module/Api/Mastodon/Instance/Extended.php
new file mode 100644 (file)
index 0000000..9849098
--- /dev/null
@@ -0,0 +1,60 @@
+<?php
+/**
+ * @copyright Copyright (C) 2010-2024, the Friendica project
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ *
+ */
+
+namespace Friendica\Module\Api\Mastodon\Instance;
+
+use DateTime;
+use Friendica\App;
+use Friendica\Core\Config\Capability\IManageConfigValues;
+use Friendica\Core\L10n;
+use Friendica\Database\Database;
+use Friendica\Model\User;
+use Friendica\Module\Api\ApiResponse;
+use Friendica\Module\BaseApi;
+use Friendica\Object\Api\Mastodon\ExtendedDescription;
+use Friendica\Util\Profiler;
+use Psr\Log\LoggerInterface;
+
+/**
+ * Undocumented API endpoint that is implemented by both Mastodon and Pleroma
+ */
+class Extended extends BaseApi
+{
+       /** @var IManageConfigValues */
+       private $config;
+
+       public function __construct(\Friendica\Factory\Api\Mastodon\Error $errorFactory, App $app, L10n $l10n, App\BaseURL $baseUrl, App\Arguments $args, LoggerInterface $logger, Profiler $profiler, ApiResponse $response, Database $database, IManageConfigValues $config, array $server, array $parameters = [])
+       {
+               parent::__construct($errorFactory, $app, $l10n, $baseUrl, $args, $logger, $profiler, $response, $server, $parameters);
+
+               $this->config = $config;
+       }
+
+       /**
+        * @throws HTTPException\InternalServerErrorException
+        */
+       protected function rawContent(array $request = [])
+       {
+               $account = User::getSystemAccount();
+
+               $this->jsonExit(new ExtendedDescription(new DateTime($account['updated']), $this->config->get('config', 'info')));
+       }
+}
index 12cb1d9..4bfde07 100644 (file)
@@ -22,7 +22,6 @@
 namespace Friendica\Module\Api\Mastodon\Instance;
 
 use Friendica\Core\Protocol;
-use Friendica\Core\System;
 use Friendica\Database\DBA;
 use Friendica\Model\GServer;
 use Friendica\Module\BaseApi;
index 6bb154d..c0f2f6e 100644 (file)
 
 namespace Friendica\Module\Api\Mastodon\Instance;
 
-use Friendica\Content\Text\BBCode;
-use Friendica\Content\Text\HTML;
 use Friendica\Core\System;
-use Friendica\DI;
 use Friendica\Module\BaseApi;
 use Friendica\Network\HTTPException;
 
index 25db399..9cef709 100644 (file)
@@ -137,6 +137,9 @@ class Search extends BaseApi
        private static function searchStatuses(int $uid, string $q, string $account_id, int $max_id, int $min_id, int $limit, int $offset)
        {
                if (Network::isValidHttpUrl($q)) {
+                       if ($offset != 0) {
+                               return [];
+                       }
                        $q = Network::convertToIdn($q);
                        // If the user-specific search failed, we search and probe a public post
                        $item_id = Item::fetchByLink($q, $uid) ?: Item::fetchByLink($q);
@@ -158,6 +161,10 @@ class Search extends BaseApi
                        $table = 'post-searchindex';
                }
 
+               if (!empty($account_id)) {
+                       $condition = DBA::mergeConditions($condition, ["`author-id` = ?", $account_id]);
+               }
+
                if (!empty($max_id)) {
                        $condition = DBA::mergeConditions($condition, ["`uri-id` < ?", $max_id]);
                }
index b9adb11..a506afe 100644 (file)
@@ -42,6 +42,8 @@ class Application extends BaseDataTransferObject
        protected $redirect_uri;
        /** @var string */
        protected $website;
+       /** @var string */
+       protected $vapid_key;
 
        /**
         * Creates an application entry
@@ -49,7 +51,7 @@ class Application extends BaseDataTransferObject
         * @param array   $item
         * @throws \Friendica\Network\HTTPException\InternalServerErrorException
         */
-       public function __construct(string $name, string $client_id = null, string $client_secret = null, int $id = null, string $redirect_uri = null, string $website = null)
+       public function __construct(string $name, string $client_id = null, string $client_secret = null, int $id = null, string $redirect_uri = null, string $website = null, string $vapid_key = null)
        {
                $this->client_id     = $client_id;
                $this->client_secret = $client_secret;
@@ -57,6 +59,7 @@ class Application extends BaseDataTransferObject
                $this->name          = $name;
                $this->redirect_uri  = $redirect_uri;
                $this->website       = $website;
+               $this->vapid_key     = $vapid_key;
        }
 
        /**
index 38a0002..968ca26 100644 (file)
@@ -46,6 +46,8 @@ class Card extends BaseDataTransferObject
        protected $provider_name;
        /** @var string */
        protected $provider_url;
+       /** @var string */
+       protected $html;
        /** @var int */
        protected $width;
        /** @var int */
@@ -53,6 +55,8 @@ class Card extends BaseDataTransferObject
        /** @var string */
        protected $image;
        /** @var string */
+       protected $embed_url;
+       /** @var string */
        protected $blurhash;
        /** @var array */
        protected $history;
@@ -73,9 +77,11 @@ class Card extends BaseDataTransferObject
                $this->author_url    = $attachment['author_url'] ?? '';
                $this->provider_name = $attachment['provider_name'] ?? '';
                $this->provider_url  = $attachment['provider_url'] ?? '';
+               $this->html          = '';
                $this->width         = $attachment['width'] ?? 0;
                $this->height        = $attachment['height'] ?? 0;
                $this->image         = $attachment['image'] ?? '';
+               $this->embed_url     = '';
                $this->blurhash      = $attachment['blurhash'] ?? '';
                $this->history       = $history;
        }
diff --git a/src/Object/Api/Mastodon/ExtendedDescription.php b/src/Object/Api/Mastodon/ExtendedDescription.php
new file mode 100644 (file)
index 0000000..96b8491
--- /dev/null
@@ -0,0 +1,44 @@
+<?php
+/**
+ * @copyright Copyright (C) 2010-2024, the Friendica project
+ *
+ * @license GNU AGPL version 3 or any later version
+ *
+ * This program is free software: you can redistribute it and/or modify
+ * it under the terms of the GNU Affero General Public License as
+ * published by the Free Software Foundation, either version 3 of the
+ * License, or (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ * GNU Affero General Public License for more details.
+ *
+ * You should have received a copy of the GNU Affero General Public License
+ * along with this program.  If not, see <https://www.gnu.org/licenses/>.
+ *
+ */
+
+namespace Friendica\Object\Api\Mastodon;
+
+use Friendica\BaseDataTransferObject;
+use Friendica\Util\DateTimeFormat;
+
+/**
+ * Class ExtendedDescription
+ *
+ * @see https://docs.joinmastodon.org/entities/ExtendedDescription/
+ */
+class ExtendedDescription extends BaseDataTransferObject
+{
+       /** @var string (Datetime) */
+       protected $updated_at;
+       /** @var string */
+       protected $content;
+
+       public function __construct(\DateTime $updated_at, string $content)
+       {
+               $this->updated_at = $updated_at->format(DateTimeFormat::JSON);
+               $this->content    = $content;
+       }
+}
index 1b2529e..c52c04d 100644 (file)
@@ -254,7 +254,7 @@ return [
                        '/instance'                          => [Module\Api\Mastodon\Instance::class,                 [R::GET         ]],
                        '/instance/activity'                 => [Module\Api\Mastodon\Unimplemented::class,            [R::GET         ]], // @todo
                        '/instance/domain_blocks'            => [Module\Api\Mastodon\Unimplemented::class,            [R::GET         ]], // @todo
-                       '/instance/extended_description'     => [Module\Api\Mastodon\Unimplemented::class,            [R::GET         ]], // @todo
+                       '/instance/extended_description'     => [Module\Api\Mastodon\Instance\Extended::class,        [R::GET         ]],
                        '/instance/peers'                    => [Module\Api\Mastodon\Instance\Peers::class,           [R::GET         ]],
                        '/instance/rules'                    => [Module\Api\Mastodon\Instance\Rules::class,           [R::GET         ]],
                        '/lists'                             => [Module\Api\Mastodon\Lists::class,                    [R::GET, R::POST]],