superbalist / laravel-pubsub Goto Github PK

Hi,
With L5.4, the app()->make() does not longer support 2 arguments.
You can't pass parameters to bind() anymore.

So the package isn't compatible with this new version.
We have many choices for that, according laravel/framework#17556

Edit: Also here laravel/ideas#391

The standard answer is to "make a factory" but.. this is already one !

So, I'm to just call new Clients in the factory without the container's make method:

But we loose the pubsub.xxx_clientaliases (I personally don't care about that, but maybe someone use them ?) and tests are more difficults to do.

I've started to made the modification on local, and can PR it if you want (I'm just wondering about the tests..).

Or maybe we can find an other better way..?

Thanks

>>> app('pubsub')->publish('channel_name', 'messagesr');                                
=> null
>>> app('pubsub')->publish('channel_name', 'messagesr');
Google\Cloud\Core\Exception\ServiceException with message 'cURL error 35: Bulk data encryption algorithm failed in selected cipher suite. (see http://curl.haxx.se/libcurl/c/libcurl-errors.html)'

On the first call channel_name is created (I can see it in the console), but on the second one it fails.

In summary, I can't connect to a Kafka broker running on a host other than localhost. I can set up a SSH tunnel to tunnel 9092 traffic on the broker to localhost, then everything works fine.

This is not a firewall-related issue either as I can use the Kafka CLI tools to make the connection to that host directly.

I have checked that my configuration settings are getting into laravel-pubsub library but for some reason it always tries to connect to localhost, and then I see some errors like this:

%3|1512781383.226|FAIL|rdkafka#producer-1| [thrd:localhost:9092/0]: localhost:9092/0: Connect to ipv4#127.0.0.1:9092 failed: Connection refused
%3|1512781383.226|ERROR|rdkafka#producer-1| [thrd:localhost:9092/0]: localhost:9092/0: Connect to ipv4#127.0.0.1:9092 failed: Connection refused
%3|1512781383.228|FAIL|rdkafka#consumer-2| [thrd:localhost:9092/0]: localhost:9092/0: Connect to ipv4#127.0.0.1:9092 failed: Connection refused
%3|1512781383.228|ERROR|rdkafka#consumer-2| [thrd:localhost:9092/0]: localhost:9092/0: Connect to ipv4#127.0.0.1:9092 failed: Connection refused

Like I said, I checked and the configs for my custom host are getting where they should be.

Hi,

Looks like a promising package. I would like to use the Kafka connector, but I failed to install this package due to some weird Google api dependency:

composer require superbalist/laravel-pubsub

Using version ^2.0 for superbalist/laravel-pubsub
./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for superbalist/laravel-pubsub ^2.0 -> satisfiable by superbalist/laravel-pubsub[2.0.0].
    - superbalist/laravel-pubsub 2.0.0 requires superbalist/php-pubsub-google-cloud ^2.0 -> satisfiable by superbalist/php-pubsub-google-cloud[2.0.0, 2.0.1].
    - superbalist/php-pubsub-google-cloud 2.0.0 requires google/cloud ^0.7.0|^0.8.0|^0.9.0 -> satisfiable by google/cloud[v0.7.0, v0.7.1, v0.8.0, v0.9.0].
    - superbalist/php-pubsub-google-cloud 2.0.1 requires google/cloud ^0.7.0|^0.8.0|^0.9.0 -> satisfiable by google/cloud[v0.7.0, v0.7.1, v0.8.0, v0.9.0].
    - google/cloud v0.7.0 requires google/auth 0.10 -> satisfiable by google/auth[v0.10].
    - google/cloud v0.7.1 requires google/auth 0.10 -> satisfiable by google/auth[v0.10].
    - google/cloud v0.8.0 requires google/auth 0.10 -> satisfiable by google/auth[v0.10].
    - google/cloud v0.9.0 requires google/auth 0.10 -> satisfiable by google/auth[v0.10].
    - Conclusion: don't install google/auth v0.10

Installation failed, reverting ./composer.json to its original content.

Hope you are able to fix this. In my case I don't even use the Google dependencies.

Regards

I am getting error to Run "composer require superbalist/laravel-pubsub"

Problem 1
- Conclusion: remove laravel/framework v5.1.46
- Conclusion: don't install laravel/framework v5.1.46
- superbalist/laravel-pubsub 2.0.0 requires illuminate/console ^5.3 -> satisfiable by illuminate/console[v5.3.0, v5.3.16, v5.3.23, v5.3.4, v5.4.0, v5.4.13, v5.4.17, v5.4.9].
- superbalist/laravel-pubsub 2.0.1 requires illuminate/console ^5.3 -> satisfiable by illuminate/console[v5.3.0, v5.3.16, v5.3.23, v5.3.4, v5.4.0, v5.4.13, v5.4.17, v5.4.9].
- don't install illuminate/console v5.3.0|don't install laravel/framework v5.1.46
- don't install illuminate/console v5.3.16|don't install laravel/framework v5.1.46
- don't install illuminate/console v5.3.23|don't install laravel/framework v5.1.46
- don't install illuminate/console v5.3.4|don't install laravel/framework v5.1.46
- don't install illuminate/console v5.4.0|don't install laravel/framework v5.1.46
- don't install illuminate/console v5.4.13|don't install laravel/framework v5.1.46
- don't install illuminate/console v5.4.17|don't install laravel/framework v5.1.46
- don't install illuminate/console v5.4.9|don't install laravel/framework v5.1.46
- Installation request for laravel/framework (locked at v5.1.46, required as 5.1.*) -> satisfiable by laravel/framework[v5.1.46].
- Installation request for superbalist/laravel-pubsub ^2.0 -> satisfiable by superbalist/laravel-pubsub[2.0.0, 2.0.1].

This is a follow up issue to update the dependency, if needed, once this PR is approved/merged: Superbalist/php-pubsub-google-cloud#8