summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBrion Vibber <brion@pobox.com>2009-11-24 15:29:47 -0800
committerBrion Vibber <brion@pobox.com>2009-11-24 15:29:47 -0800
commita9bbfd1852077e6ddf26efec88002c0c6d4bb072 (patch)
tree1e771af5e74a816ead4508f57aced7560311682e
parente0ae89867b5caf19b12aa510195632d021d51d6e (diff)
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.
-rw-r--r--lib/omb.php2
1 files changed, 1 insertions, 1 deletions
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)