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

Use true JID of message sender when we know it #1178

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

singpolyma
Copy link
Contributor

Save true counterpart JID of MUC sender when we know it.

Set true counterpart JID by XEP-0033 ofrom, but only when it is quite safe, that is when the domain of the ofrom matches the domain of the from, and that domain advertises support for XEP-0033 itself. (Allowing for fallback body in conjunction with this.)

Show nickname and icon based on true counterpart JID when we know it.

Instead of always trying to infer from MUC presences later.
Set true counterpart JID by XEP-0033 ofrom, but only when it is quite safe, that
is when the domain of the ofrom matches the domain of the from, and that domain
advertises support for XEP-0033 itself.

When we do this, also allow for a fallback body.
@Ppjet6
Copy link
Contributor

Ppjet6 commented Mar 19, 2023

In which context does Movim do 0033? (TIL).

I would suggest not to use the realjid when we're not whois = anyone and we're a moderator. I don't know if it's comparable but 392 does this, bases the color on realjids, and it's actually annoying seeing colors change for the same person in different rooms just because we have different permissions.

Also, not much we can do as long as there's no way to do validation, but this is probably going to come bite us at some point: $presence->mucjid != $this->jidfrom.'/'.$this->resource

@edhelas
Copy link
Member

edhelas commented Mar 19, 2023

I still don't know exactly why this PR is for, I'll try to figure it out 🤔

@singpolyma
Copy link
Contributor Author

singpolyma commented Mar 19, 2023 via email

@singpolyma
Copy link
Contributor Author

singpolyma commented Mar 19, 2023 via email

@singpolyma
Copy link
Contributor Author

Just following up here with respect to the "more info" label -- happy to provide more info if needed :)

@ItsRogueRen
Copy link

Just adding my comments here as well, I have chats where I'd need to see my local name for a contact so I can tell who is who in a group of say coworkers who all have weird usernames

@nabzord
Copy link

nabzord commented Jun 4, 2023

+1 merge this

@Neustradamus
Copy link
Contributor

Any news on this PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

6 participants