coyote-team / coyote-drupal

0 stars 0 forks source link

GuzzleHTTP 7.x dependency problem #16

Closed jkva closed 1 year ago

jkva commented 1 year ago

Drupal9's internal GuzzleHTTP version requirement conflicts with the GuzzleHTTP required by CoyoteApiClient 1.0.22. Currently, this is bypassed by locally patching Drupal's vendor/guzzlehttp/guzzle/src/functions.php and changing the line

$defaultAgent = 'GuzzleHttp/' . Client::VERSION;

into

$defaultAgent = 'GuzzleHttp/' . '6.5.5';

Ideally this should be resolved CoyoteApiClient-side by either scoping its Guzzle dependency or verifying tests run fine on Guzzle 6.x . Scoping using php-scoper would in this case be preferable.

jkva commented 1 year ago

This was addressed via https://github.com/coyote-team/coyote-drupal/commit/f8fffa2f706f4d20ca5358739b1ac4d1b866ca23