#2348365: Update to Guzzle 5 introduced react/promise library 2.1. Version 2.2 is out, and includes a bug fix for this issue. Without this fix, fatal errors can result if bootstrapping Drupal from another system that has already loaded the promise library.

CommentFileSizeAuthor
#1 update-react-promise-2442411-01-no-test.txt1.66 KBjhedstrom
#1 update-react-promise-2442411-01.patch99.58 KBjhedstrom

Comments

jhedstrom’s picture

jhedstrom
Primary language English
Location Portland, OR
commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new update-react-promise-2442411-01.patch99.58 KB
new update-react-promise-2442411-01-no-test.txt1.66 KB

Attached is a patch, and a txt file patch that only contains the changes to composer.lock for easier review.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

Can't be bad, but its a bit odd that we can't update our own composer.json file that way.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Confirmed that the patch is adding nothing unexpected. Committed 6743175 and pushed to 8.0.x. Thanks!

  • alexpott committed 6743175 on 8.0.x 
    Issue #2442411 by jhedstrom: Update react/promise to 2.2.0

Status: Fixed » Closed (fixed)

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