Merge pull request #3 from andriusbaliutis/master

Fix issue with statistics class property redirect url

Author: Bonscho

README.md

@@ -26,7 +26,7 @@ Find out more at the [„PSR-7 is imminent, and here's my issues with it“](htt
 ```{json}
 {
    	"require": {
-        "chroma-x/basic-http-client": "~3.0"
+        "chroma-x/basic-http-client": "~4.0"
     }
 }
 ```

composer.json

@@ -10,13 +10,18 @@
 		"Basic Auth",
 		"Client Certificate Auth"
 	],
-	"homepage": "http://chroma-x.de/",
+	"homepage": "https://chroma-x.de/",
 	"license": "MIT",
 	"authors": [
 		{
 			"name": "Martin Brecht-Precht",
 			"email": "mb@chroma-x.de",
-			"homepage": "http://chroma-x.de"
+			"homepage": "https://chroma-x.de"
+		},
+		{
+			"name": "Andrius Baliutis",
+			"email": "ab@chroma-x.de",
+			"homepage": "https://chroma-x.de"
 		}
 	],
 	"autoload": {

src/Response/Statistics/Statistics.php

@@ -26,7 +26,7 @@ class Statistics
 
 	private float $redirectTime;
 
-	private string $redirectEndpoint;
+	private ?string $redirectEndpoint = null;
 
 	public function populateFromCurlResult(\CurlHandle|false $curl): self
 	{
@@ -40,70 +40,49 @@ public function populateFromCurlResult(\CurlHandle|false $curl): self
 		$this->startTransferTime = curl_getinfo($curl, CURLINFO_STARTTRANSFER_TIME);
 		$this->redirectCount = curl_getinfo($curl, CURLINFO_REDIRECT_COUNT);
 		$this->redirectTime = curl_getinfo($curl, CURLINFO_REDIRECT_TIME);
-		$this->redirectEndpoint = curl_getinfo($curl, CURLINFO_REDIRECT_URL);
+		$redirectUrl = curl_getinfo($curl, CURLINFO_REDIRECT_URL);
+		if ($redirectUrl !== false) {
+			$this->redirectEndpoint = $redirectUrl;
+		}
 		return $this;
 	}
 
-	/**
-	 * @return float
-	 */
 	public function getTotalTime(): float
 	{
 		return $this->totalTime;
 	}
 
-	/**
-	 * @return float
-	 */
 	public function getHostLookupTime(): float
 	{
 		return $this->hostLookupTime;
 	}
 
-	/**
-	 * @return float
-	 */
 	public function getConnectionEstablishTime(): float
 	{
 		return $this->connectionEstablishTime;
 	}
 
-	/**
-	 * @return float
-	 */
 	public function getPreTransferTime(): float
 	{
 		return $this->preTransferTime;
 	}
 
-	/**
-	 * @return float
-	 */
 	public function getStartTransferTime(): float
 	{
 		return $this->startTransferTime;
 	}
 
-	/**
-	 * @return int
-	 */
 	public function getRedirectCount(): int
 	{
 		return $this->redirectCount;
 	}
 
-	/**
-	 * @return float
-	 */
 	public function getRedirectTime(): float
 	{
 		return $this->redirectTime;
 	}
 
-	/**
-	 * @return string
-	 */
-	public function getRedirectEndpoint(): string
+	public function getRedirectEndpoint(): ?string
 	{
 		return $this->redirectEndpoint;
 	}