I can't catch a HTTP Client exception

@senrab you need a try catch wrapper. Do you have it?

foreach ($listsToCheck->get() as $listToCheck) {
	try{
		$response = Http::get($listToCheck->url);
	} catch (\Exception $e)
	{
		// handle the exception
	}
              
}

I did, however I will try again :)

Thanks @laracoft, that got me a little further.

Where would I use $response->throw() in that catch() as I won't have access to $response there?

@senrab

Which info do you want from $response? The docs don't say much, but if it was a 400 or 500 error, it won't throw exceptions, you check for them right after the Http::get() https://laravel.com/docs/8.x/http-client#error-handling

In your case of an invalid domain, that info can be read off Illuminate\Http\Client\ConnectionException, there won't be any headers or body, so a $response is of no use.

You have access to the URL via $listToCheck->url.

I was hoping to get $response->status() at least so I can log that in the database as a failed attempt.

The docs also promise $response->serverError() or $response->clientError(), but I don't seem to have access to them after the exception.

Pretty sure guzzle would give me the status code after an exception.

@senrab

1. ->status() usually refers to the HTTP code returned by the server.
2. If your case, the server did not response at all. There isn't any ->serverError().
3. Anyway, where did you see ->serverError() and ->clientError()?
4. I checked the Facade, vendor\laravel\framework\src\Illuminate\Http\Client\Factory.php, it does not provide much.
5. You could use guzzle directly (That's what I'm doing in my own Laravel 5/6 code)

@senrab those functions are to be used this way:

foreach ($listsToCheck->get() as $listToCheck) {
    try {
        { // option A
            $response = Http::get($listToCheck->url);

            // Determine if the status code was >= 200 and < 300...
            if (!$response->successful())
            {
                $response->throw();
            }
        }

        { // or option B
            $response = Http::get($listToCheck->url);

            // Determine if the status code was >= 400...
            $c1 = $response->failed();

            // Determine if the response has a 400 level status code...
            $c2 = $response->clientError();

            // Determine if the response has a 500 level status code...
            $c3 = $response->serverError();

            if ($c1 or $c2 or $c3)
                $response->throw();
        }
    } catch (\Exception $e)
    {
	// handle other exceptions such as ConnectionException
    }
}

One Way to do this, is use your own custom Exception and pass the response reason phrase.

 if ($response->successful()) {
            return $response->json();

        }

        if ($response->failed()) {
            throw new LoginFailedException($response->getReasonPhrase());
        };

chech this link, #Error Handling :https://laravel.com/docs/8.x/http-client