mirror of https://github.com/zulip/zulip.git
7f196967ad
A few internal fields used for tracking which types of notifications have already been sent for a given message, like `hander_id` and the `push_notified` bundle of fields were being incorrectly included in message events delivered to clients clients. One could argue these fields might be useful hints to clients, but because notifications can be triggered later on via `missedmessage_hook`, they have no useful purpose in the API. This commit move these extended event field on a `internal_data` object within the event object, and delete this field in `contents()` for call points that would serve data to clients. Tweaked by tabbott to provide a cleaner interface. We're not bumping API_FEATURE_LEVEL because these fields have always been documented as being present only due to a bug, so no clients should be expecting or relying on them. Fixes: #15947. |
||
|---|---|---|
| .. | ||
| __init__.py | ||
| application.py | ||
| autoreload.py | ||
| descriptors.py | ||
| django_api.py | ||
| event_queue.py | ||
| exceptions.py | ||
| handlers.py | ||
| ioloop_logging.py | ||
| sharding.py | ||
| views.py | ||