HomePhorge

Fix HTTP 400 from notification server for JSON subscription objects

Description

Fix HTTP 400 from notification server for JSON subscription objects

Summary: Fixes T5651. Sometime we'll send an object to the notification server for subscribers, which it will choke on. Use array_values() to make sure we're sending an array.

Test Plan: With (object) instead, got a consistent error ("no .filter method on object"). With array_values(), no error.

Reviewers: joshuaspence

Reviewed By: joshuaspence

Subscribers: epriestley

Maniphest Tasks: T5651

Differential Revision: https://secure.phabricator.com/D9963

Details

Provenance
epriestleyAuthored on Jul 17 2014, 2:48 PM
themackabuPushed on Tue, Mar 25, 8:07 PM
Parents
rPab3c17a2cd97: Emit more usable results from phrequent.tracking
Branches
Unknown
Tags
Unknown

Event Timeline