From: Brion Vibber <brion@pobox.com>
Date: Tue, 24 Nov 2009 23:29:47 +0000 (-0800)
Subject: Fix regression in OMB sending with queues enabled -- items never got dequeued, leadin... 
X-Git-Url: https://git.mxchange.org/?a=commitdiff_plain;h=4b59cf0e3f4acc688a513c4cd31e8eb8c74e708c;p=quix0rs-gnu-social.git

Fix regression in OMB sending with queues enabled -- items never got dequeued, leading to ever-growing queue and big delays.

Success return code from omb_broadcast_message was dropped in commit ec88d2650ea4371cf53229171851747b31587e4b (Aug 10 2009) which switched us to libomb backend. With queues enabled, this would lead to the notice being readded to the outgoing OMB queue for redelivery as the queue system thought the send failed. The resends caused extra load and confusion for third-party sites, and more worryingly just plugged up our own queue so legit messages were badly delayed.

This commit should restore the previous state, where we fire-and-forget; that is, we're not actually checking to see if all remote subscribers received the message successfully and there will be no resends.
---

diff --git a/lib/omb.php b/lib/omb.php
index cd6d6e1b25..49496b774e 100644
--- a/lib/omb.php
+++ b/lib/omb.php
@@ -102,7 +102,7 @@ function omb_broadcast_notice($notice)
         common_debug('Finished to ' . $rp->postnoticeurl, __FILE__);
     }
 
-    return;
+    return true;
 }
 
 function omb_broadcast_profile($profile)