Page MenuHomePhabricator

Fix Feed Notification Handler for Phabot.
Needs RevisionPublic

Authored by matelich on May 15 2015, 9:57 PM.
Tags
None
Referenced Files
F15436491: D12867.id.diff
Tue, Mar 25, 2:35 PM
F15429727: D12867.id30951.diff
Mon, Mar 24, 3:54 AM
F15418890: D12867.id30951.diff
Fri, Mar 21, 2:07 AM
F15413865: D12867.diff
Wed, Mar 19, 9:47 PM
Unknown Object (File)
Feb 20 2025, 8:31 PM
Unknown Object (File)
Feb 16 2025, 9:18 AM
Unknown Object (File)
Feb 9 2025, 6:33 PM
Unknown Object (File)
Jan 31 2025, 7:27 AM

Details

Summary

Related to T7829. feed.query's after/before parameters seems to be feed-relative, not time-relative. Before a chrono key means later in time (at least by my testing). The notification.channels setting was not being used, all joined channels were receiving messages. If a large number of messages come in at once, the bot may be disconnected for filling RecvQ of the IRC server. Added a sleep of increasing length, similar to behavior of IRC clients and bulk pastes. Add documentation of notifications.verbosity

Test Plan

manual testing

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 6019
Build 6039: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

matelich retitled this revision from to Fix Feed Notification Handler for Phabot..
matelich updated this object.
matelich edited the test plan for this revision. (Show Details)
matelich added a reviewer: epriestley.
matelich added a subscriber: eadler.
eadler added a reviewer: eadler.

This seems reasonable to me.

src/infrastructure/daemon/bot/handler/PhabricatorBotFeedNotificationHandler.php
175

eh, ++ inside a sleep statement seems a bit magical

epriestley edited edge metadata.

See T7829.

This revision now requires changes to proceed.May 25 2015, 2:48 PM