Support PSR HTTP client

[garak]

It would be nice to not restrict HTTP client usage to Guzzle, and using a PSR HTTP client instead

[dereuromark]

Looks like this could be included in https://github.com/yidas/google-maps-services-php/pull/8 right?

[yidas]

It looks like in #8, the composer is still using Guzzle.

I think it's inconvenient to maintain a PHP client without a mainstream HTTP client package.

[garak]

@yidas you can still include Guzzle of course, but without forcing other people to use it

[yidas]

Hi @garak,

As I know, PSR-18 is the definition of the client interface. May I know more about how you actually want to achieve?

[garak]

I'd like to use this library without being forced to use Guzzle. This would also improve the quality of the code, making it more SOLID.

[yidas]

Got it. It can be a goal, and once it's implemented we can remove Guzzle dependency.

[alex-kalanis]

Possible with #8 -> instead of the whole client (facade there) just call inner processings

// ...
# today
$calls = new \yidas\googleMaps\Services(new \yidas\googleMaps\Clients\GuzzleClient(), new \yidas\googleMaps\Services\ServiceFactory(new \yidas\googleMaps\ApiAuth($optParams)));
# via PSR
$calls = new \yidas\googleMaps\Services(new \yidas\googleMaps\Clients\PsrClient($yourPsrClient), new \yidas\googleMaps\Services\ServiceFactory(new \yidas\googleMaps\ApiAuth($optParams)));
// ...

[garak]

Not a good idea: dependencies should be injected, not instantiated (again, to be SOLID)

[dereuromark]

I think the main issue remaining is

    "require": {
        ...
        "guzzlehttp/guzzle": "~6.5.6|^7.4.3"
    },

There should be a solution to require the abstract only, not the concrete. my guess: https://packagist.org/packages/psr/http-client That's probably what he mainly means.

If we have to major now anyways, with that PR being merged, maybe this is the chance to also clean up that dependency issue

[alex-kalanis]

@garak : So you just call

class Something
{
    public function __construct(
        protected \yidas\googleMaps\Services $service
    ) {}
}

And set the rest of dependency tree somewhere else... Like in neon config

parameters:
  apiKey: YOUR_API_KEY

services:
  - yidas\googleMaps\ApiAuth(%apiKey%)
  - yidas\googleMaps\Services\ServiceFactory
  - yidas\googleMaps\Clients\PsrClient
  - yidas\googleMaps\Services

I see no problem.

[garak]

@alex-kalanis I was talking about the relationship between \yidas\googleMaps\Services and the HTTP client

[dereuromark]

To which I added context that clarifies it further :)

"require": {
    ...
    "psr/http-client": "^1.0.3"
},

and move guzzle to require-dev for test harness (as default)

[garak]

I'm not sure to understand how it was clarified... the only relevant subject here is the constructor of the class receiving the HTTP client

[dereuromark]

No, also the compose require part as outlined in my previous comment here :) Otherwise you gain nothing really.

[garak]

That can be relevant elsewhere, just not relevant here.