-
Notifications
You must be signed in to change notification settings - Fork 370
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
MSC3754: Removing profile information #3754
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Kévin Commaille <zecakeh@tedomum.fr>
Signed-off-by: Kévin Commaille <zecakeh@tedomum.fr>
To identify the change, the `m.room.member` and `m.presence` events sent following the call to one of these two | ||
endpoints MUST include the changed key with a `null` value. Subsequent `m.room.member` events SHOULD omit the deleted | ||
key altogether. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why must the m.room.member
eventsinclude null
instead of omitting the values? The current spec for m.room.member
and m.presence
for these fields is pretty clear:
The {avatar URL|display name} for this user, if any.
My reading of this is that the new values fully replace any old values and omitted values should be cleared.
Note that I think https://spec.matrix.org/v1.10/client-server-api/#events-on-change-of-profile-information is the useful part of the spec we're talking about? Cross-linking in the MSC is very much appreciated.
The `m.room.member` event omits the changed key in `content`. This means that all the keys that are not unset should be | ||
present every time a part of the profile is changed. This is limiting for custom keys as they might be omitted by | ||
clients or servers that don't recognize them. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not really following this, maybe fewer double negatives helps?
The `m.room.member` event omits the changed key in `content`. This means that all the keys that are not unset should be | |
present every time a part of the profile is changed. This is limiting for custom keys as they might be omitted by | |
clients or servers that don't recognize them. | |
The `m.room.member` event omits the changed key in `content`. This means that all the keys that ar set should be | |
present every time a part of the profile is changed. This is limiting for custom keys as they might be omitted by | |
clients or servers that don't recognize them. |
Rendered
Related spec issues: