- Issue created by @leymannx
- leymannx Berlin
Uh, seems my project only has Guzzle 6.5.8. And seems that config_patch_gitlab_api requires Guzzle 7
- leymannx Berlin
Uff, so drupal/core-recommended is on "guzzlehttp/guzzle": "~6.5.8"
https://github.com/drupal/core-recommended/blob/9.5.x/composer.json#L17
- leymannx Berlin
Okay, so
composer require php-http/guzzle6-adapter
fixed this for me. Not sure, should this be part of the docs? Or better in the composer.json? And what's with projects that are better off withguzzle7-adapter
? - 🇩🇪Germany a.dmitriiev
The core itself has dependency to either one of the version https://github.com/drupal/core/blob/9.5.x/composer.json#L39:
"guzzlehttp/guzzle": "^6.5.8 || ^7.4.5",
But the problem is in adapter itself, because it requires
Psr\Http\Client\ClientInterface
interface. - 🇩🇪Germany a.dmitriiev
And Drupal 10 supports only Guzzle 7, so I think that was a problem. Now either adapter should be installed manually, or a new version of the module should be released specially for Drupal 10
- @admitriiev opened merge request.
- 🇩🇪Germany a.dmitriiev
Please review MR, it adds back the guzzle6 adapter and removes Drupal 10 support, because of incompatible guzzle version. For Drupal 10 the 3.x version of the module will be needed.
- 🇩🇪Germany a.dmitriiev
3.x branch was opened https://www.drupal.org/project/config_patch_gitlab_api/releases/3.x-dev →
- Status changed to Needs review
over 1 year ago 1:16pm 26 April 2023 - Status changed to RTBC
over 1 year ago 1:46pm 9 May 2023 - 🇩🇪Germany breidert
I tested it with a custom installation of Drupal. After installing the forked repository and requiring php-http/guzzle6-adapter I could configure credential on the administration UI of the module without errors.
Moving to RTBC
-
a.dmitriiev →
committed 88a24cb1 on 2.x
Issue #3343691: No PSR-18 clients found error
-
a.dmitriiev →
committed 88a24cb1 on 2.x
- Status changed to Fixed
over 1 year ago 5:02pm 9 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.