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

XEP-0421 v0.3.0 #1413

Merged
merged 1 commit into from
Jan 8, 2025
Merged

XEP-0421 v0.3.0 #1413

merged 1 commit into from
Jan 8, 2025

Conversation

mar-v-in
Copy link
Contributor

@mar-v-in mar-v-in commented Jan 8, 2025

  • Adjust wording to use semi-anonymous or pseudonymous instead of anonymous
  • Explicitly mention issues arising from occupant id matching across rooms
  • Add example with server secret instead of room secret
  • Add some pseudocode

Diff is here: https://larma.de/xeps/xep-0421-diff.html
Note that as part of not using the term anonymous anymore, I also adjusted the title to make it more descriptive.

- Adjust wording to use semi-anonymous or pseudonymous instead of anonymous
- Explicitly mention issues arising from occupant id matching across rooms
- Add example with server secret instead of room secret
- Add some pseudocode
@github-actions github-actions bot added the Needs Council The affected XEP has the Council as Approving Body and it needs to decide on the change. label Jan 8, 2025
@mar-v-in
Copy link
Contributor Author

mar-v-in commented Jan 8, 2025

This PR addresses feedback from the list. I don't think council is needed for the author to incorporate list feedback during Proposed status, but this case wasn't handled in the triage script. I created a PR to adjust the triage script to treat Proposed the same as Experimental. See #1414

@iNPUTmice iNPUTmice merged commit 56b2cec into xsf:master Jan 8, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Council The affected XEP has the Council as Approving Body and it needs to decide on the change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants