Problem/Motivation

In Drupal 8 and 9, Drupal had an indirect dependency on guzzlehttp/psr7 because Guzzle uses it internally to create Request and Response objects.

In Drupal 10, numerous core services depend on guzzlehttp/psr7 as our PSR-17 implementation, so we should add a direct dependency and constraint for it.

Proposed resolution

Add a direct dependency on the guzzlehttp/psr7 package.

Remaining tasks

TBD

User interface changes

N/A

API changes

N/A

Data model changes

N/A

Release notes snippet

Drupal core now has a direct dependency on the guzzlehttp/psr7 package (previously an indirect dependency) since it is used for the default implementation of several core services for PSR-17.

Issue fork drupal-3270882

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:

Comments

xjm created an issue. See original summary.

xjm’s picture

Issue summary: View changes

xjm’s picture

Status: Active » Needs review
xjm’s picture

xjm’s picture

Issue summary: View changes
spokje’s picture

Status: Needs review » Reviewed & tested by the community

MR:

- Does what it says on the tin: Adding guzzlehttp/psr7 as a direct dependency
- TestBot likes it

RTBC for me.

dww’s picture

Agreed, MR looks good. Do we do change records for things like this?

Thanks,
-Derek

dww’s picture

Issue tags: +Bug Smash Initiative

p.s. This seems more like a task to me. But since @xjm says it's a bug, let's smash it. 😀

  • catch committed 5e03f38 on 10.0.x
    Issue #3270882 by xjm: Drupal 10 uses guzzlehttp/psr7 for PSR-17...
catch’s picture

Status: Reviewed & tested by the community » Fixed

I added this issue to the existing CR: https://www.drupal.org/node/3256584

Committed 5e03f38 and pushed to 10.0.x. Thanks!

xjm’s picture

Issue tags: +10.0.0 release notes

Status: Fixed » Closed (fixed)

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