Problem/Motivation

Since we support only Guzzle 7 in Drupal 10 we can catch \Psr\Http\Client\ClientExceptionInterface errors instead of \GuzzleHttp\Exception\GuzzleException and take a small step towards making the client used by Drupal swappable.

Follow-up from #3189301: Use \Psr\Http\Client\ClientExceptionInterface instead of \GuzzleHttp\Exception\TransferException

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

Issue fork drupal-3418243

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

Spokje created an issue. See original summary.

Spokje opened merge request !6401

spokje’s picture

spokje
Primary language Dutch
commented
Status: Active » Needs review
smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Searched repo for GuzzleHttp\Exception\GuzzleException and found 2 instances which were covered by MR.

  • catch committed 47475efa on 11.x 
    Issue #3418243 by Spokje, smustgrave: Use \Psr\Http\Client\...
catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 11.x, thanks!

Spokje closed merge request !6401

Status: Fixed » Closed (fixed)

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