search

pierobot / mangapie

This is a self-hosted server for archived manga.
BSD 3-Clause "New" or "Revised" License
65 stars 9 forks source link

Time out while refreshing Library #203

Closed tgaugry closed 4 years ago

tgaugry commented 4 years ago

Hello, I've been trying to add my manga library through the web interface. Unfortunately, clicking on the refresh button always end up with a 504 Gateway Time-out. Associated php error : [28-May-2020 02:56:14] WARNING: [pool www] child 56192 said into stderr: "NOTICE: PHP message: PHP Fatal error: Maximum execution time of 30 seconds exceeded in /usr/local/www/mangapie/vendor/laravel/framework/src/Illuminate/Database/Eloquent/Model.php on line 387"

I've tried the same procedure with and without the commands php artisan queue:listen, php artisan queue:work and php artisan schedule:run running in the background. Didn't see any difference.

Is it due to the size of my library (500+) or did I miss something in the documentation ?

Edit: I managed to find the mangapie:scan function and imported everything. I still can't however refresh the collection.

pierobot commented 4 years ago

The solution is to use php artisan queue:work --timeout=0. I've updated the wikis. :+1:

tgaugry commented 4 years ago

Still the same problem. I'm assuming that is not using the queue, even if launched. Any way to check that ?

pierobot commented 4 years ago

In your queue worker, you should see text that says Processing when it's working a queue item and Processed once it finishes.

pierobot@pierobot:~/github/mangapie$ php artisan queue:work
[2020-06-02 21:01:34][RNk4w5onowHR7I7DNGE1eg3vU62EpZiR] Processing: App\Notifications\NewArchiveNotification
[2020-06-02 21:01:34][RNk4w5onowHR7I7DNGE1eg3vU62EpZiR] Processed:  App\Notifications\NewArchiveNotification

If you don't see anything, then there's probably an issue with the queue driver. What do you have QUEUE_DRIVER set to in your .env file? Admittedly, I haven't tried it with drivers other than redis.

When you refresh a library using the admin libraries page, does the progress bar move?

tgaugry commented 4 years ago

That was that; the queue driver was set to sync. Changing it to redis solved the problem. The documentation only mentionned the cache driver, so i only changed that.

Thanks !

pierobot commented 4 years ago

No problem, hope you find the app useful. Feel free to open issues for questions or suggestions. I know it might be lacking in some areas.