Skip to content

Commit

Permalink
Merge pull request #157 from laravel/chore/avoids-upgrade-of-guzzle-p…
Browse files Browse the repository at this point in the history
…romises

[2.x] Avoids non-tested update of guzzle promises 2.x
  • Loading branch information
nunomaduro authored Aug 18, 2023
2 parents 36886e9 + 72b8092 commit 61adb35
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,8 @@
"require": {
"php": "^7.2|^8.0",
"aws/aws-sdk-php": "^3.80",
"psr/http-message": "^1.0",
"guzzlehttp/promises": "^1.4.0",
"guzzlehttp/guzzle": "^6.3|^7.0",
"hollodotme/fast-cgi-client": "^3.0",
"illuminate/container": "^6.0|^7.0|^8.0|^9.0|^10.0",
Expand Down

2 comments on commit 61adb35

@matt-travis
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I keep getting the following error when I try to deploy:
{ "exception": { "class": "Error", "message": "Call to undefined function GuzzleHttp\Promise\settle()", "code": 0, "file": "/var/task/vendor/laravel/vapor-core/src/Runtime/Handlers/WarmerHandler.php:25" } }

Would you be able to help?

@nunomaduro
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matt-travis Well, this pull request fixes exactly that issue. Anyways, can you try to run composer update laravel/vapor-core --with-all-dependencies ?

Please sign in to comment.