fix(CrawlService): Use IClient instead of raw Guzzle

lib/Activity/Filter.php

@@ -27,7 +27,6 @@ public function __construct(IL10N $l, IURLGenerator $urlGenerator) {
 		$this->urlGenerator = $urlGenerator;
 	}
 
-
 	/**
 	 * @inheritDoc
 	 */

lib/BackgroundJobs/EmptyTrashbinJob.php

@@ -17,7 +17,6 @@ class EmptyTrashbinJob extends TimedJob {
 	public const INTERVAL = 5 * 60; // 5 minutes
 	public const TRASHBIN_TTL = 2 * 4 * 4 * 7 * 24 * 60 * 60; // Two months
 
-
 	public function __construct(
 		ITimeFactory $timeFactory,
 		private TreeMapper $treeMapper,

lib/Controller/BookmarkController.php

@@ -390,7 +390,6 @@ public function newBookmark(string $url = '', ?string $title = null, ?string $de
 		}
 	}
 
-
 	/**
 	 * @throws UnauthenticatedError
 	 */
@@ -692,7 +691,6 @@ public function exportBookmark() {
 		return new ExportResponse($data);
 	}
 
-
 	/**
 	 * @throws UnauthenticatedError
 	 */

lib/Controller/FoldersController.php

@@ -287,7 +287,6 @@ public function undeleteFromFolder(int $folderId, int $bookmarkId): JSONResponse
 		}
 	}
 
-
 	/**
 	 * @throws UnauthenticatedError
 	 */

lib/Controller/InternalFoldersController.php

@@ -37,15 +37,12 @@ public function addFolder(string $title = '', int $parent_folder = -1): JSONResp
 		return $this->controller->addFolder($title, $parent_folder);
 	}
 
-
-
 	#[NoAdminRequired]
 	#[FrontpageRoute(verb: 'GET', url: '/folder/{folderId}/childorder')]
 	public function getFolderChildrenOrder(int $folderId, int $layers = 0): JSONResponse {
 		return $this->controller->getFolderChildrenOrder($folderId, $layers);
 	}
 
-
 	#[NoAdminRequired]
 	#[FrontpageRoute(verb: 'PATCH', url: '/folder/{folderId}/childorder')]
 	public function setFolderChildrenOrder(int $folderId, array $data = []): JSONResponse {
@@ -106,7 +103,6 @@ public function getFolderPublicToken(int $folderId): DataResponse {
 		return $this->controller->getFolderPublicToken($folderId);
 	}
 
-
 	#[NoAdminRequired]
 	#[FrontpageRoute(verb: 'POST', url: '/folder/{folderId}/publictoken')]
 	public function createFolderPublicToken(int $folderId): DataResponse {
@@ -137,7 +133,6 @@ public function findShares(): DataResponse {
 		return $this->controller->findShares();
 	}
 
-
 	#[NoAdminRequired]
 	#[FrontpageRoute(verb: 'POST', url: '/folder/{folderId}/shares')]
 	public function createShare(int $folderId, string $participant, int $type, bool $canWrite = false, bool $canShare = false): DataResponse {

lib/Controller/WebViewController.php

@@ -65,7 +65,6 @@ public function __construct(
 		parent::__construct($appName, $request);
 	}
 
-
 	#[NoAdminRequired]
 	#[NoCSRFRequired]
 	#[FrontpageRoute(verb: 'GET', url: '/')]

lib/Dashboard/Frequent.php

@@ -33,7 +33,6 @@ public function __construct(IL10N $l, \OCP\IURLGenerator $url) {
 		$this->url = $url;
 	}
 
-
 	/**
 	 * @inheritDoc
 	 */

lib/Dashboard/Recent.php

@@ -33,7 +33,6 @@ public function __construct(IL10N $l, \OCP\IURLGenerator $url) {
 		$this->url = $url;
 	}
 
-
 	/**
 	 * @inheritDoc
 	 */

lib/Db/BookmarkMapper.php

@@ -183,7 +183,6 @@ protected function mapRowToEntity(array $row): Bookmark {
 		return call_user_func($this->entityClass . '::fromRow', $row);
 	}
 
-
 	/**
 	 * Find a specific bookmark by Id
 	 *
@@ -782,7 +781,6 @@ public function findAllInPublicFolder(string $token, QueryParameters $queryParam
 			->from('*PREFIX*bookmarks', 'b')
 			->innerJoin('b', 'folder_tree', 'tree', 'tree.item_id = b.id AND tree.type = ' . $qb->createPositionalParameter(TreeMapper::TYPE_BOOKMARK) . ' AND tree.soft_deleted_at is NULL');
 
-
 		$this->_filterUrl($qb, $queryParams);
 		$this->_filterArchived($qb, $queryParams);
 		$this->_filterUnavailable($qb, $queryParams);

lib/Db/Folder.php

@@ -31,7 +31,6 @@ class Folder extends Entity {
 
 	public static $columns = ['id', 'title', 'user_id'];
 
-
 	public function __construct() {
 		// add types in constructor
 		$this->addType('title', 'string');

lib/Db/FolderMapper.php

@@ -34,7 +34,6 @@ class FolderMapper extends QBMapper {
 	 */
 	protected $sharedFolderMapper;
 
-
 	/**
 	 * @var ShareMapper
 	 */
@@ -115,7 +114,6 @@ public function findRootFolder(string $userId): Folder {
 		return $rootFolder;
 	}
 
-
 	/**
 	 * @param Entity $entity
 	 * @psalm-param Folder $entity

lib/Db/TreeMapper.php

@@ -54,7 +54,6 @@ class TreeMapper extends QBMapper {
 
 	protected $entityColumns = [];
 
-
 	private IQueryBuilder $insertQuery;
 
 	private IQueryBuilder $parentQuery;
@@ -371,7 +370,6 @@ public function hasDescendant(int $folderId, string $type, int $descendantId): b
 		return true;
 	}
 
-
 	/**
 	 * @param string $type
 	 * @psalm-param TreeMapper::TYPE_* $type
@@ -690,7 +688,6 @@ public function move(string $type, int $itemId, int $newParentFolderId, ?int $in
 		));
 	}
 
-
 	/**
 	 * @brief Add a bookmark to a set of folders
 	 * @param string $type
@@ -1034,7 +1031,7 @@ public function countChildren(int $folderId): int {
 			->from('bookmarks_tree')
 			->where($qb->expr()->eq('parent_folder', $qb->createPositionalParameter($folderId)))
 			->andWhere($qb->expr()->isNull('soft_deleted_at'));
-		return $qb->executeQuery()->fetch(PDO::FETCH_COLUMN);
+		return (int)$qb->executeQuery()->fetch(PDO::FETCH_COLUMN);
 	}
 
 	/**

lib/Flow/Bookmark.php

@@ -66,7 +66,6 @@ public function __construct(IL10N $l, IURLGenerator $urlGenerator, BookmarkMappe
 		$this->authorizer = $authorizer;
 	}
 
-
 	/**
 	 * @inheritDoc
 	 */

lib/QueryParameters.php

@@ -64,7 +64,6 @@ public function setTags(array $tags): self {
 		return $this;
 	}
 
-
 	/**
 	 * @return int
 	 */

lib/Service/Authorizer.php

@@ -37,7 +37,6 @@ class Authorizer {
 
 	private $cors = false;
 
-
 	public function __construct(
 		private FolderMapper $folderMapper,
 		private BookmarkMapper $bookmarkMapper,

lib/Service/BookmarksParser.php

@@ -74,7 +74,6 @@ class BookmarksParser {
 	 */
 	private $ignorePersonalToolbarFolder = true;
 
-
 	/**
 	 * Constructor
 	 *

lib/Service/CirclesService.php

@@ -8,7 +8,6 @@
 
 declare(strict_types=1);
 
-
 namespace OCA\Bookmarks\Service;
 
 use OCP\App\IAppManager;

lib/Service/CrawlService.php

@@ -11,8 +11,6 @@
 use Exception;
 use fivefilters\Readability\Configuration;
 use fivefilters\Readability\Readability;
-use GuzzleHttp\Client;
-use GuzzleHttp\Psr7\Response;
 use Mimey\MimeTypes;
 use OC\User\NoUserException;
 use OCA\Bookmarks\Db\Bookmark;
@@ -24,6 +22,9 @@
 use OCP\Files\IRootFolder;
 use OCP\Files\NotFoundException;
 use OCP\Files\NotPermittedException;
+use OCP\Http\Client\IClient;
+use OCP\Http\Client\IClientService;
+use OCP\Http\Client\IResponse;
 use OCP\IConfig;
 use OCP\IL10N;
 use OCP\Lock\LockedException;
@@ -36,18 +37,21 @@ class CrawlService {
 	public const READ_TIMEOUT = 30;
 	public const UA_FIREFOX = 'Mozilla/5.0 (X11; Linux x86_64; rv:150.0) Gecko/20100101 Firefox/150.0';
 
+	private IClient $client;
 	private MimeTypes $mimey;
 
 	public function __construct(
 		private BookmarkMapper $bookmarkMapper,
 		private BookmarkPreviewer $bookmarkPreviewer,
 		private FaviconPreviewer $faviconPreviewer,
+		IClientService $clientService,
 		private IConfig $config,
 		private IRootFolder $rootFolder,
 		private IL10N $l,
 		private LoggerInterface $logger,
 		private UserSettingsService $userSettingsService,
 	) {
+		$this->client = $clientService->newClient();
 		$this->mimey = new MimeTypes;
 	}
 
@@ -60,9 +64,7 @@ public function crawl(Bookmark $bookmark): void {
 			return;
 		}
 		try {
-			$client = new Client();
-			/** @var Response $resp */
-			$resp = $client->get($bookmark->getUrl(), [
+			$resp = $this->client->get($bookmark->getUrl(), [
 				'headers' => [
 					'User-Agent' => self::UA_FIREFOX,
 				],
@@ -91,15 +93,13 @@ public function crawl(Bookmark $bookmark): void {
 		$this->bookmarkMapper->update($bookmark);
 	}
 
-	private function archiveContent(Bookmark $bookmark, Response $resp): void {
-		$header = $resp->getHeader('Content-Type');
+	private function archiveContent(Bookmark $bookmark, IResponse $resp): void {
+		$contentType = $resp->getHeader('Content-Type');
 
-		if (empty($header)) {
+		if ($contentType === '') {
 			return;
 		}
 
-		$contentType = $header[0] ?? null;
-
 		if ($contentType !== null && str_contains($contentType, 'text/html')) {
 			if ($bookmark->getHtmlContent() === null || $bookmark->getHtmlContent() === '') {
 				$config = new Configuration();
@@ -120,18 +120,16 @@ private function archiveContent(Bookmark $bookmark, Response $resp): void {
 		}
 	}
 
-	private function archiveFile(Bookmark $bookmark, Response $resp): void {
-		$header = $resp->getHeader('Content-Type');
+	private function archiveFile(Bookmark $bookmark, IResponse $resp): void {
+		$contentType = $resp->getHeader('Content-Type');
 
-		if (empty($header)) {
+		if ($contentType === '') {
 			return;
 		}
 
-		$contentType = $header[0] ?? null;
-
 		if ($contentType !== null && !str_contains($contentType, 'text/html') && $bookmark->getArchivedFile() === null) {
 			$contentLengthHeader = $resp->getHeader('Content-Length');
-			$contentLength = isset($contentLengthHeader[0]) ? (int)$contentLengthHeader[0] : 0;
+			$contentLength = $contentLengthHeader !== '' ? (int)$contentLengthHeader : 0;
 
 			if ($contentLength < self::MAX_BODY_LENGTH) {
 				try {

lib/Service/FolderService.php

@@ -328,7 +328,6 @@ public function createShare($folderId, $participant, int $type, bool $canWrite =
 			$this->addSharedFolderForParticipant($share, $folder, $type, $participant);
 		}
 
-
 		return $share;
 	}
 

lib/Service/HtmlImporter.php

@@ -37,7 +37,6 @@ class HtmlImporter {
 
 	private int $transactionCounter = 0;
 
-
 	public function __construct(
 		private BookmarkMapper $bookmarkMapper,
 		private FolderMapper $folderMapper,

lib/Service/Previewers/GenericUrlBookmarkPreviewer.php

@@ -33,7 +33,6 @@ class GenericUrlBookmarkPreviewer implements IBookmarkPreviewer {
 	 */
 	private $client;
 
-
 	/** @var LoggerInterface */
 	private $logger;
 

lib/Service/Previewers/ScreeenlyBookmarkPreviewer.php

@@ -33,7 +33,6 @@ class ScreeenlyBookmarkPreviewer implements IBookmarkPreviewer {
 	 */
 	private $client;
 
-
 	/** @var LoggerInterface */
 	private $logger;
 

lib/Service/Previewers/ScreenshotMachineBookmarkPreviewer.php

@@ -46,7 +46,6 @@ class ScreenshotMachineBookmarkPreviewer implements IBookmarkPreviewer {
 	 */
 	private $height = 800;
 
-
 	public function __construct(IConfig $config, IClientService $clientService, LoggerInterface $logger) {
 		$this->apiKey = $config->getAppValue('bookmarks', 'previews.screenshotmachine.key', '');
 		$this->client = $clientService->newClient();

lib/Service/Previewers/WebshotBookmarkPreviewer.php

@@ -111,8 +111,6 @@ public function fetchImage($url): ?Image {
 			return null;
 		}
 
-
-
 		return new Image('image/jpeg', $body);
 	}
 }

lib/Service/TreeCacheManager.php

@@ -79,7 +79,6 @@ public function __construct(
 		$this->caches[self::CATEGORY_CHILDORDER] = $cacheFactory->createDistributed('bookmarks:' . self::CATEGORY_CHILDORDER);
 	}
 
-
 	private function getTreeMapper(): TreeMapper {
 		return $this->appContainer->get(TreeMapper::class);
 	}

psalm-baseline.xml

@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<files psalm-version="6.14.2@bbd217fc98c0daa0a13aea2a7f119d03ba3fc9a0">
+<files psalm-version="6.16.1@f1f5de594dc76faf8784e02d3dc4716c91c6f6ac">
   <file src="lib/AppInfo/Application.php">
     <InvalidArgument>
       <code><![CDATA[UsersGroupsCirclesListener::class]]></code>
@@ -106,14 +106,10 @@
   </file>
   <file src="lib/Service/CrawlService.php">
     <UndefinedClass>
-      <code><![CDATA[Client]]></code>
-      <code><![CDATA[Response]]></code>
-      <code><![CDATA[Response]]></code>
+      <code><![CDATA[$e]]></code>
+      <code><![CDATA[$e]]></code>
+      <code><![CDATA[NoUserException]]></code>
     </UndefinedClass>
-    <UndefinedDocblockClass>
-      <code><![CDATA[$resp]]></code>
-      <code><![CDATA[$resp]]></code>
-    </UndefinedDocblockClass>
   </file>
   <file src="lib/Service/FolderService.php">
     <InvalidReturnStatement>

tests/BookmarkMapperTest.php

@@ -1,6 +1,5 @@
 <?php
 
-
 namespace OCA\Bookmarks\Tests;
 
 use OC;