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

Question to changelog: Adds Horizon & Redis for jobs, cache. #46

Closed
mauthi opened this issue Feb 16, 2018 · 6 comments
Closed

Question to changelog: Adds Horizon & Redis for jobs, cache. #46

mauthi opened this issue Feb 16, 2018 · 6 comments

Comments

@mauthi
Copy link
Contributor

mauthi commented Feb 16, 2018

Is there a documentation available how to configure this?
I never used horizon before.
Is this a replacement for the jobs table? Is it possible to still use the jobs table and php artisan queue:work?
If I run this command and I start a download I get the file but there is a job for notfication 255 times executed after that.

Sorry for my questions but this topic is new for me.

@aocneanu
Copy link
Member

I see now that the ChangeLog is misleading. This update is only for the live demo and it does not affect other projects. The queue / cache driver is set in the .env file, so as long as you use database / sync for the queue, and file / database for the cache you are fine.

@mauthi
Copy link
Contributor Author

mauthi commented Feb 16, 2018

ok, fine!

My settings in .env:

CACHE_DRIVER=file
QUEUE_DRIVER=database

But after a download I get the following output on console:

[2018-02-16 09:22:35][515] Processing: LaravelEnso\VueDatatable\app\Jobs\ExcelExport
[2018-02-16 09:22:37][515] Processed:  LaravelEnso\VueDatatable\app\Jobs\ExcelExport
[2018-02-16 09:22:37][516] Processing: Illuminate\Notifications\Events\BroadcastNotificationCreated
[2018-02-16 09:22:38][517] Processing: Illuminate\Notifications\Events\BroadcastNotificationCreated
[2018-02-16 09:22:38][518] Processing: Illuminate\Notifications\Events\BroadcastNotificationCreated
[2018-02-16 09:22:38][519] Processing: Illuminate\Notifications\Events\BroadcastNotificationCreated
[2018-02-16 09:22:38][520] Processing: Illuminate\Notifications\Events\BroadcastNotificationCreated
[2018-02-16 09:22:38][521] Processing: Illuminate\Notifications\Events\BroadcastNotificationCreated
[2018-02-16 09:22:38][522] Processing: Illuminate\Notifications\Events\BroadcastNotificationCreated
...
...

And no notfication is created.

In browser console I get the following:
WebSocket connection to 'ws://ws.pusherapp.com/app/958ac7a43a4ac76cfabf?protocol=7&client=js&version=4.2.2&flash=false' failed: WebSocket is closed before the connection is established.

@aocneanu
Copy link
Member

this in indeed due to a change in l5.6 broadcasting.php config file.

before we had under the connections.options.cluster key the eu value. Now it's a new env variable PUSHER_APP_CLUSTER.

So just add PUSHER_APP_CLUSTER=eu to your .env

@mauthi
Copy link
Contributor Author

mauthi commented Feb 16, 2018

Thx!

@mauthi mauthi closed this as completed Feb 16, 2018
@mauthi
Copy link
Contributor Author

mauthi commented Feb 16, 2018

Was this in changelog?

@aocneanu
Copy link
Member

Upgrades Laravel to 5.6 :). All the files from the local project are updated too.

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

2 participants