Override of Guzzle request default options

[bab-uk-rob]

There does not appear to be a way to do this.

It's possible to set a different HttpClient but the $options passed passed on to HttpClient::request() are built based on Client::getDefaultParams() with no easy way to adjust.

It should be possible to extend the Client class and override Client::getDefaultParams() but it seems there is nothing built in for this.

[pkrzaczkowski-hippo]

Hey

Why not just pass Your own Guzzle HttpClient by calling

/**
     * Sets GuzzleHttp client.
     * @param BaseClient $client
     */
    public function setClient(BaseClient $client): void
    {
        $this->httpClient = $client;
    }

[pkrzaczkowski-hippo]

Hey,

Oooo now I see where the problem is

protected function getDefaultParams($endpoint): array

Sure - i don't know when I will have time to address this - but PR's are welcome :)

Pawel

[bab-uk-rob]

Hi thanks for looking at this. I know what you mean about available time. If I can find some, I'll put together a PR. For now, I have had to use a helper method for a work-around to get the Client which looks like this and seems to be working:

private function getCustomerIoClient(): Client
{
    $client = new class (
        '__API_KEY__',
        '__SITE_ID__,
        ['region' => 'eu']
    ) extends Client {
        protected function getDefaultParams($endpoint): array
        {
            $params = parent::getDefaultParams($endpoint);
            $params['connect_timeout'] = 5; // <-- our connect timeout
            $params['timeout'] = 10; // <-- our request timeout
            return $params;
        }
    };

    $client->setAppAPIKey('__APP_API_KEY__');

    return $client;
}

[pkrzaczkowski-hippo]

If it's only about timeout - maybe for now we could just expose a setting to modify the default ones - simpler to implement - rather than allowing full custom options for the client.