#184 - the Configuration object is used – XpathCheck

app/Check/Consumers/Check.php

@@ -61,7 +61,6 @@ public function process(\PhpAmqpLib\Message\AMQPMessage $message): int
 
 
 	/**
-	 * @param \Pd\Monitoring\Check\Check $check
 	 * @return bool TRUE, pokud se podařilo úspěšně provést kontrolu, jinak FALSE. Po FALSE může následovat opětovné spuštění kontroly
 	 */
 	abstract protected function doHardJob(\Pd\Monitoring\Check\Check $check): bool;

app/Check/Consumers/XpathCheck.php

@@ -10,24 +10,17 @@ final class XpathCheck extends Check
 	private ?string $lastUrl = NULL;
 
 
-	/**
-	 * @param \Pd\Monitoring\Check\Check|\Pd\Monitoring\Check\XpathCheck $check
-	 * @return bool
-	 */
 	protected function doHardJob(\Pd\Monitoring\Check\Check $check): bool
 	{
+		if ( ! $check instanceof \Pd\Monitoring\Check\XpathCheck) {
+			throw new \Pd\Monitoring\Exception();
+		}
 		$check->xpathLastResult = NULL;
 
-		$client = new \GuzzleHttp\Client();
-
-		$options = [
-			'connect_timeout' => $check::ALIVE_TIMEOUT / 1000,
-			'timeout' => 2 * $check::ALIVE_TIMEOUT / 1000,
-			'headers' => [
-				'User-Agent' => 'PeckaMonitoringBot/1.0',
-			],
-			'allow_redirects' => TRUE,
-		];
+		$guzzleOptions = \Pd\Monitoring\Check\Consumers\Client\Configuration::create($check::ALIVE_TIMEOUT / 1000, (int) (\round(2 * ($check::ALIVE_TIMEOUT / 1000))))
+			->withAllowRedirects(\Pd\Monitoring\Check\Consumers\Client\Configuration\AllowRedirects::create(TRUE))
+		;
+		$client = new \GuzzleHttp\Client($guzzleOptions->config());
 
 		if ($check->siteMap && ! $this->siteMapLoader) {
 			$this->siteMapLoader = new \Pd\Monitoring\Check\SiteMapLoader($check->url);
@@ -37,7 +30,7 @@ protected function doHardJob(\Pd\Monitoring\Check\Check $check): bool
 			if ($this->siteMapLoader) {
 				$withoutError = TRUE;
 				while ($url = $this->siteMapLoader->getNextUrl($this->lastUrl)) {
-					$loaded = $this->loadUrl($client, $options, $check, $url);
+					$loaded = $this->loadUrl($client, $check, $url);
 					if ( ! $loaded) {
 						$this->logError($check, \sprintf('Došlo k chybě na url "%s"', $url));
 						$withoutError = FALSE;
@@ -47,7 +40,7 @@ protected function doHardJob(\Pd\Monitoring\Check\Check $check): bool
 
 				return $withoutError;
 			} else {
-				$loaded = $this->loadUrl($client, $options, $check, $check->url);
+				$loaded = $this->loadUrl($client, $check, $check->url);
 				if ($loaded) {
 					return TRUE;
 				}
@@ -67,9 +60,9 @@ protected function getCheckType(): int
 	}
 
 
-	private function loadUrl(\GuzzleHttp\Client $client, array $options, \Pd\Monitoring\Check\XpathCheck $check, string $url): bool
+	private function loadUrl(\GuzzleHttp\Client $client, \Pd\Monitoring\Check\XpathCheck $check, string $url): bool
 	{
-		$response = $client->request('GET', $url, $options);
+		$response = $client->get($url);
 
 		if ($response->getStatusCode() !== 200) {
 			$this->logHeaders($check, $response);

phpstan-baseline.neon

@@ -455,26 +455,11 @@ parameters:
 			count: 1
 			path: app/Check/Consumers/RabbitQueueCheck.php
 
-		-
-			message: "#^Access to an undefined property Pd\\\\Monitoring\\\\Check\\\\Check\\:\\:\\$xpathLastResult\\.$#"
-			count: 1
-			path: app/Check/Consumers/XpathCheck.php
-
-		-
-			message: "#^Access to undefined constant Pd\\\\Monitoring\\\\Check\\\\Check\\:\\:ALIVE_TIMEOUT\\.$#"
-			count: 2
-			path: app/Check/Consumers/XpathCheck.php
-
 		-
 			message: "#^Cannot call method getNextUrl\\(\\) on Pd\\\\Monitoring\\\\Check\\\\SiteMapLoader\\|null\\.$#"
 			count: 1
 			path: app/Check/Consumers/XpathCheck.php
 
-		-
-			message: "#^Method Pd\\\\Monitoring\\\\Check\\\\Consumers\\\\XpathCheck\\:\\:loadUrl\\(\\) has parameter \\$options with no value type specified in iterable type array\\.$#"
-			count: 1
-			path: app/Check/Consumers/XpathCheck.php
-
 		-
 			message: "#^Only booleans are allowed in a negated boolean, Pd\\\\Monitoring\\\\Check\\\\SiteMapLoader\\|null given\\.$#"
 			count: 1
@@ -485,11 +470,6 @@ parameters:
 			count: 1
 			path: app/Check/Consumers/XpathCheck.php
 
-		-
-			message: "#^Parameter \\#3 \\$check of method Pd\\\\Monitoring\\\\Check\\\\Consumers\\\\XpathCheck\\:\\:loadUrl\\(\\) expects Pd\\\\Monitoring\\\\Check\\\\XpathCheck, Pd\\\\Monitoring\\\\Check\\\\Check given\\.$#"
-			count: 2
-			path: app/Check/Consumers/XpathCheck.php
-
 		-
 			message: "#^Only booleans are allowed in &&, string\\|null given on the left side\\.$#"
 			count: 1