Skip to content
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

Unclear what a membership event means in the case of changing global profile info #2086

Open
TheArcaneBrony opened this issue Feb 25, 2025 · 1 comment
Labels
clarification An area where the expected behaviour is understood, but the spec could do with being more explicit

Comments

@TheArcaneBrony
Copy link

Link to problem area:
https://spec.matrix.org/v1.13/client-server-api/#events-on-change-of-profile-information

Issue
Noticed that some homeservers don't clear the reason field of a membership when updating one's global display name.

After some discussion, some homeserver implementation maintainers and I agreed that the spec is ambiguous on what to do with other fields in the membership events.

Should this be a new event? Should servers automatically clear some fields (reason, is_direct, join_authorised_by_users_server, etc.)? How does this work with custom profile fields (ie. room level overrides for custom profile keys)?

@TheArcaneBrony TheArcaneBrony added the clarification An area where the expected behaviour is understood, but the spec could do with being more explicit label Feb 25, 2025
@richvdh
Copy link
Member

richvdh commented Feb 25, 2025

Noticed that some homeservers don't clear the reason field of a membership when updating one's global display name.

Not really following. Presumably, if you're updating your display name, the previous membership event was a join. Why would a join event have a reason property?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
clarification An area where the expected behaviour is understood, but the spec could do with being more explicit
Projects
None yet
Development

No branches or pull requests

2 participants