-
Notifications
You must be signed in to change notification settings - Fork 401
MSC3672: Sharing ephemeral streams of location data #3672
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
stefanceriu
wants to merge
21
commits into
main
Choose a base branch
from
stefan/ephemeral-location-streaming
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+97
−0
Open
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
12066dd
MSC3672: Sharing ephemeral streams of location data
stefanceriu 55fb0ed
Referencing MSCs through PR link instead of rendered.
stefanceriu 861dc4c
Link to encrypted EDUs MSC.
stefanceriu c7c032e
Add to-device alternative.
stefanceriu 2a26b20
Add more details on EDUs and use cases.
stefanceriu f439d3e
typo
anoadragon453 fb30996
Describe each field in a table, clean up example
anoadragon453 f063795
Clarify m.beacon_info, m.beacon are now defined by this MSC
anoadragon453 440a5d1
Mention that the current design does not support multi-share
anoadragon453 a72bda3
Note MSCs that we depend on
anoadragon453 de543b1
Rename Encryption section to Security considerations, and add another
anoadragon453 6144b23
Add top-level fields to m.beacon EDU example
anoadragon453 35c20f6
Delete a paragraph that didn't make sense
anoadragon453 a62d237
Clarify some client UI suggestions from multiple shares
anoadragon453 76df8b5
Clarify m.beacon example
anoadragon453 4a922c4
Remove paragraph that was already made clear by MSC2477
anoadragon453 f035694
Remove superfluous inner m.beacon_info dict
anoadragon453 e07abb3
swap type and key columns in the field definition tables
anoadragon453 45389a5
Clean up field definitions, add table for m.beacon
anoadragon453 43d224a
Clarify that m.ts is in milliseconds
anoadragon453 750635d
Remodel to lean heavily on MSC3489, instead of repeating
andybalaam File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,97 @@ | ||
# MSC3672 - Sharing ephemeral streams of location data | ||
|
||
## Problem | ||
|
||
[MSC3489](https://github.com/matrix-org/matrix-doc/pull/3489) | ||
focuses on streaming persistent location data for applications that require | ||
historical knowledge. | ||
|
||
While that's perfect for situations in which long term storage of the data is a | ||
non-issue and implied (e.g. flight paths, strava style exercises, fleet | ||
tracking), there are also cases in doing so is undesirable for either privacy | ||
or performance reasons. | ||
|
||
Sharing user live location updates is one of the cases in which privacy is | ||
paramount and where we need the ability to share data in a safe and | ||
non-persistent fashion. | ||
|
||
## Proposal | ||
|
||
This MSC adds the ability to publish short-lived location beacons through | ||
the use of Ephemeral Data Units (EDUs). It is almost identical to MSC3489, | ||
apart from this change. | ||
|
||
Ephemeral data units (EDUs) are Matrix's default mechanism for broadcasting | ||
short-lived data to a group of users. They are intended to be | ||
non-persistent, not take part in a room's history and are currently used | ||
for typing notifications, event receipts, and presence updates. | ||
|
||
To allow the use of EDUs for live location sharing, this MSC depends on | ||
[MSC2477](https://github.com/matrix-org/matrix-spec-proposals/pull/2477/) | ||
for user-defined EDUs, and | ||
[MSC3673](https://github.com/matrix-org/matrix-spec-proposals/pull/3673) | ||
for encrypted EDUs. | ||
|
||
Clients announce a share, and stop a share exactly as in MSC3489, by | ||
emitting an `m.beacon_info` state event. | ||
|
||
Clients share locations by emitting an `m.beacon` event which is identical | ||
to those described in MSC3489, except that it is an EDU (whereas in MSC3489 | ||
the events are PDUs). | ||
|
||
All other behaviour is the same as in MSC3489. The only difference is that | ||
`m.beacon` events are EDUs. | ||
|
||
### Delivery guarantees | ||
|
||
MSC2477 specifically avoids defining delivery guarantees for user-defined EDUs, | ||
so we avoid making hard recommendations here. We expect that future MSCs will | ||
clarify the available options. When that happens, clients should choose the | ||
available option that suits their use case, which we expect will be broadly in | ||
line with the following paragraph. | ||
|
||
Servers should attempt to deliver `m.beacon` EDUs to all clients, and delete | ||
them as soon as possible. Servers should implement a timeout mechanism to ensure | ||
they are always deleted, even if undelivered. For the expected use cases, we | ||
suggest 30 minutes would be a suitable timeout. | ||
|
||
## Security considerations | ||
|
||
The security considerations are identical to MSC3489, except that EDUs | ||
provide a better privacy situation because they are not persisted long-term. | ||
|
||
## Alternatives | ||
|
||
Alternatively, we could negotiate a WebRTC data channel using | ||
[MSC3401](https://github.com/matrix-org/matrix-doc/pull/3401) | ||
and stream low-latency geospatial data over the participating devices in the | ||
room. However it would be useful to support plain HTTP like the rest of Matrix | ||
and requiring a WebRTC stack is prohibitively complicated for simple clients | ||
(e.g. basic IOT devices reporting spatial telemetry). | ||
|
||
Another alternative is to use to-device events but that comes with disadvantages | ||
of its own as they're 1:1, single message per transaction and not intended for | ||
conversational data. | ||
|
||
## Dependencies | ||
|
||
This proposal relies on the following MSCs: | ||
|
||
* [MSC2477: User-defined ephemeral events in rooms](https://github.com/matrix-org/matrix-spec-proposals/pull/2477) | ||
* [MSC3673: Encrypted ephemeral data units](https://github.com/matrix-org/matrix-spec-proposals/pull/3673) | ||
* [MSC3489: Sharing streams of location data with history](https://github.com/matrix-org/matrix-spec-proposals/pull/3489) | ||
|
||
For live location shares to work in appservices, these MSCs will also be | ||
required: | ||
|
||
* [MSC2409: Proposal to send EDUs to appservices](https://github.com/matrix-org/matrix-spec-proposals/pull/2409) | ||
* [MSC3202: Encrypted appservices](https://github.com/matrix-org/matrix-spec-proposals/pull/3202) | ||
|
||
## Unstable prefix | ||
|
||
This MSC uses the same unstable prefixes as | ||
[MSC3489](https://github.com/matrix-org/matrix-spec-proposals/pull/3489). | ||
|
||
Note that MSC3489 actually uses prefixes that match this MSC's numbering. | ||
This is an historical artifact of the various revision histories of these | ||
proposals. |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.