search

thephpleague / oauth1-client

OAuth 1 Client
MIT License
968 stars 73 forks source link

Remove deprecated Guzzle function call #135

Closed bencorlett closed 3 years ago

bencorlett commented 3 years ago

The GuzzleHttp\Psr7\uri_for function is deprecated and removed in Guzzle 7.2. Where available GuzzleHttp\Psr7\Utils::uriFor is used instead.

Given we support Guzzle 6 and 7, both the method and function call are required to be supported.

Fixes #134.

masakik commented 3 years ago

Hi, I have this issue yesterday and tested this PR. Seems to be a good solution.

hawkinbj commented 3 years ago

Have the same issue, tested this PR and it solves the problem as well

bencorlett commented 3 years ago

@masakik and @hawkinbj thank you both for testing! Incoming release.

GrahamCampbell commented 3 years ago

function is deprecated and removed in Guzzle 7.2

This is not true. Guzzle follows semantic versioning.

GrahamCampbell commented 3 years ago

See https://github.com/thephpleague/oauth1-client/pull/136. :)

bencorlett commented 3 years ago

@GrahamCampbell if you would like, feel welcome to submit a PR