eileenmcnaughton / org.wikimedia.geocoder

Geocoder for CiviCRM
Other
6 stars 17 forks source link

1.12: No PSR-18 clients found on Civi 5.74.3 #63

Open chrisgaraffa opened 2 weeks ago

chrisgaraffa commented 2 weeks ago

60 mentions that 1.12 should work on recent Civi (5.69+), but we're getting the same error after updating to 5.74.3 and Geocoder 1.12:

Got error 'PHP message: Uncaught PHP Exception Drupal\Core\Entity\EntityStorageException: "No PSR-18 clients found. Make sure to install a package providing "psr/http-client-implementation". Example: "php-http/guzzle7-adapter"." at /srv/www/SITE/web/core/lib/Drupal/Core/Entity/Sql/SqlContentEntityStorage.php line 815',

composer show -i guzzlehttp/guzzle shows Guzzle 6.5.8 is installed

$ composer why guzzlehttp/guzzle
civicrm/civicrm-core      5.74.3  requires guzzlehttp/guzzle (^6.3 || ^7.3)     
drupal/core               9.5.11  requires guzzlehttp/guzzle (^6.5.8 || ^7.4.5) 
drupal/core-recommended   9.5.11  requires guzzlehttp/guzzle (~6.5.8)           
drush/drush               11.6.0  requires guzzlehttp/guzzle (^6.5 || ^7.0)     
enlightn/security-checker v1.10.0 requires guzzlehttp/guzzle (^6.3|^7.0)        
league/oauth2-client      2.7.0   requires guzzlehttp/guzzle (^6.0 || ^7.0)

Drupal 9.5.11 PHP 8.1.2-1ubuntu2.17

eileenmcnaughton commented 1 week ago

do I read this correctly as saying that drupal 9 core is restricting the guzzle version to 6.x

drupal/core-recommended 9.5.11 requires guzzlehttp/guzzle (~6.5.8)

chrisgaraffa commented 1 week ago

Sorry, got busy deploying the Civi security release. Yes, that's correct.

Reviewing this more, we probably shouldn't be using drupal/core and drupal/core-recommended on the same site. Not sure how common this setup is, but it's probably not "recommended" or supported. Sites using core-recommended would still be impacted, though D9 is out of security support at this point too.