HttpClient is deprecated

I will work on this issue.

I had replace the depreciated code for HTTPClient and also solve some coding issues with the branch.
kindly review the MR.

Thanks a lot. This was committed into the 8.x-4.x-dev branch, is going to be part of next Geocoder release

itamair → closed merge request !22

With this changes now unable to install site with Drupal 10.0.7.

[error] Error: Cannot instantiate interface Psr\Http\Client\ClientInterface in Drupal\Component\DependencyInjection\Container->createService() (line 259 of /home/runner/work/acquia_cms/orca-build/docroot/core/lib/Drupal/Component/DependencyInjection/Container.php) #0 /home/runner/work/acquia_cms/orca-build/docroot/core/lib/Drupal/Component/DependencyInjection/Container.php(177): Drupal\Component\DependencyInjection\Container->createService()

Please see attached screenshots

Nice catch @Rajeshreeputra ... super thanks for this. Reopening this then.

vishalkhode → made their first commit to this issue’s fork.

I was too quick (and confident) with the initial fixes / contributions to this issue (that also affected the 3.x branch).
Please provide a full QA and Testing of what is being done here, so that I could perform mine one, being confident this properly marked as RTBC ...
This has become a super adopted module, so that any critical regression introduced would reflect in a big mess, coming from everywhere.

To avoid these type of issue in future lets add testcase coverage in separate ticket.

Automatically closed - issue fixed for 2 weeks with no activity.

itamair → closed merge request !24