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

/invite API does not fill displayname field #3492

Open
t3chguy opened this issue Jan 8, 2025 · 0 comments
Open

/invite API does not fill displayname field #3492

t3chguy opened this issue Jan 8, 2025 · 0 comments
Labels
T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.

Comments

@t3chguy
Copy link
Member

t3chguy commented Jan 8, 2025

Background information

  • Dendrite version or git SHA: dendrite-demo-pinecone:main
  • SQLite3 or Postgres?: sqlite3
  • Running in Docker?: yes
  • go version: dendrite-demo-pinecone:main
  • Client used (if applicable): element-web

Description

  • What is the problem: DM rooms with pending invites show up with a userId instead of display name
  • Who is affected: Most people seemingly
  • How is this bug manifesting: /sync returns less helpful data
  • When did this first appear: Unclear

Steps to reproduce

  • Invite an existing user with a display name
  • Observe your invite event has no displayname field

The spec does not explicitly say to do this, maybe it ought to. Synapse does this. Without this invites are significantly more confusing to the end user given that they may have invited via the User Directory graphically, so never saw the MXID, now only see the MXID until the user joins.

SynapseTrace.zip
DendriteTrace.zip

@S7evinK S7evinK added the T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues. label Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-Defect Bugs, crashes, hangs, security vulnerabilities, or other reported issues.
Projects
None yet
Development

No branches or pull requests

2 participants