element-hq / element-hq/synapse

Unnecessary `device_lists.changed` entry in /sync after fetching device list

Open
#13,783 0 comments 0 reactions 0 assignees View on GitHub
A-Device-List-Tracking A-Sync O-Occasional S-Tolerable T-Defect
Dominant language
Python
Stars
4.6k
Forks
600
Avg merge
5d 22h
Merged PRs (30d)
51

Description

This issue has been migrated from [#13783](https://github.com/matrix-org/synapse/issues/13783).

---

While writing https://github.com/matrix-org/complement/pull/459, it was noticed that Synapse would sometimes emit an unnecessary `device_lists.changed` entry in the next sync after retrieving the device list for a remote user.

While it doesn't contravene the spirit of the spec, it's inefficient as well-behaved clients will ask the homeserver for device keys a second time.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.