From 4b59cf0e3f4acc688a513c4cd31e8eb8c74e708c Mon Sep 17 00:00:00 2001 From: Brion Vibber Date: Tue, 24 Nov 2009 15:29:47 -0800 Subject: 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. --- lib/omb.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/omb.php b/lib/omb.php index cd6d6e1b2..49496b774 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) -- cgit v1.2.3-54-g00ecf