Don't fork worker on frontend by default

diff --git a/mod/item.php b/mod/item.php
index ac5b11c..51bbc76 100644 (file)
--- a/mod/item.php
+++ b/mod/item.php
@@ -835,7 +835,7 @@ function item_post(App $a) {
        // When we are doing some forum posting via ! we have to start the notifier manually.
        // These kind of posts don't initiate the notifier call in the item class.
        if ($only_to_forum) {
-               Worker::add(PRIORITY_HIGH, "Notifier", Delivery::POST, $post_id);
+               Worker::add(['priority' => PRIORITY_HIGH, 'dont_fork' => false], "Notifier", Delivery::POST, $post_id);
        }
 
        Logger::log('post_complete');

diff --git a/src/Core/Worker.php b/src/Core/Worker.php
index 225a928..236f8b4 100644 (file)
--- a/src/Core/Worker.php
+++ b/src/Core/Worker.php
@@ -1125,7 +1125,8 @@ class Worker
                }
 
                $priority = PRIORITY_MEDIUM;
-               $dont_fork = Config::get("system", "worker_dont_fork", false);
+               // Don't fork from frontend tasks by default
+               $dont_fork = Config::get("system", "worker_dont_fork", false) || !\get_app()->isBackend();
                $created = DateTimeFormat::utcNow();
                $force_priority = false;