[SENTRY-2036] sentry_sync_notifications() should set ID when it returns errors

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.0.0
• Fix Version/s: 2.0.0
• Component/s: Sentry
• Assignee: Alex Kolbasov
• Reporter: Alex Kolbasov
• Created: 07/Nov/17 18:13
• Updated: 09/Nov/17 00:01
• Resolved: 09/Nov/17 00:01

Description

In sentry_sync_notifications() function, when it receives an exception, it doesn't set ID in the rsponse. SInce ID is mandatory field, it later causes Thrift exception on ID which isn't set. We should always set it in the reply.

spena FYI