Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error caused by guzzle psr-7 update #1661

Closed
rajeshjayan60 opened this issue Jun 6, 2019 · 4 comments
Closed

Error caused by guzzle psr-7 update #1661

rajeshjayan60 opened this issue Jun 6, 2019 · 4 comments
Assignees
Labels
type: question Request for information or clarification. Not an issue.

Comments

@rajeshjayan60
Copy link

rajeshjayan60 commented Jun 6, 2019

Environment details

  • OS: Ubuntu
  • PHP version: 5.6
  • Package name and version: Google api client 2.2.3

Error

Header value must be a string or an array of strings.

Code

$token = $client->fetchAccessTokenWithAuthCode($code);

The latest update to Guzzle psr7 guzzle/psr7#250, causes an error with google api client when trying to fetch an access token.

To fix it I had to downgrade the Guzzle psr7 version to 1.5.2 in composer.json

"guzzlehttp/psr7": "1.5.2"
@agwidarsito
Copy link

Thanks rajeshjayan60, I too am getting errors but I'm yet to identify which of the packages is using Guzzle in a way that's causing it.

@agwidarsito
Copy link

Update: This breaks the aws/aws-sdk-php package version 3.99.2 S3 client

@jdpedrie
Copy link
Contributor

jdpedrie commented Jun 6, 2019

v1.5.2 is the latest release of guzzlehttp/psr7, so I'm not sure what you are downgrading from. Are you using dev-master or another unreleased version of the dependency? We don't recommend relying on unreleased package versions when it can be avoided.

It appears that they've merged a fix for this already: guzzle/psr7#272

@jdpedrie jdpedrie added the type: question Request for information or clarification. Not an issue. label Jun 6, 2019
@rajeshjayan60
Copy link
Author

Thanks for the reply. It seems my guzzlehttp/psr7 dependency was pointing to dev-master. Changing the minimum-stability to stable in composer.json seems to do the trick.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: question Request for information or clarification. Not an issue.
Projects
None yet
Development

No branches or pull requests

3 participants