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

Broken if "kriswallsmith/buzz" > 0.16.1 #74

Open
maxixcom opened this issue May 31, 2018 · 3 comments
Open

Broken if "kriswallsmith/buzz" > 0.16.1 #74

maxixcom opened this issue May 31, 2018 · 3 comments

Comments

@maxixcom
Copy link

It seems there are some changes in Buzz API(https://github.com/kriswallsmith/Buzz) since 0.17. That's why transmission-php is broken now. It fails with message about missing class Buzz\Message\Request now.
But everything is fine in versions 0.16.x. Let's make restriction up to this version.

@candasm
Copy link

candasm commented Nov 7, 2018

Also because of curl send method which is deprecated in buzz package client class, setClient prompts an deprecated error.

candasm added a commit to candasm/transmission-php that referenced this issue Nov 7, 2018
I will like to fix issue kleiram#74 with locking buzz package version.
@divinity76
Copy link

a backwards-compatible quick-fix is to simply give the connecting host address to Client in the constructor,

eg:

$auth = new Client("example.org");

instead of

$auth = new Client();

@ParitoshBh
Copy link

Made changes to use version 0.15.2 of buzz dependency. Here's the forked repo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants