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

Ghost sessions #298

Open
Zash opened this issue Sep 28, 2021 · 2 comments
Open

Ghost sessions #298

Zash opened this issue Sep 28, 2021 · 2 comments
Labels
S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems.

Comments

@Zash
Copy link

Zash commented Sep 28, 2021

Observation: Many of copies of the same stanza like <message type="groupchat" from="#[email protected]" to="[email protected]/RANDOM_VARYING">

Theory: When disconnecting, sometimes the <presence type="unavailable"/> does not make it to the bridge. If the server policy is to generate a random resource (the JID resource part is a session identifier, not a device identifier!) then these eventually add up in MUCs and the like if not removed.

Background: In XMPP / MUC, clients that silently go offline without the MUC being told end up as "ghosts", appearing to be still present but aren't. These are normally removed when an attempt to deliver a groupchat message to them results in an error returned to the MUC. These kinds of problems were common in XMPP long ago, but were mostly fixed.

Solution: Watch for <message type="error"> in response to groupchat messages, check whether the error condition points to non-deliverability (e.g. one of these) and then remove the user.

Might be a duplicate of #53

@Half-Shot Half-Shot added the T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems. label May 18, 2022
@Half-Shot
Copy link
Collaborator

I think this definitely warrants further investigation.

@Half-Shot Half-Shot added the S-Major Severely degrades major functionality or product features, with no satisfactory workaround label May 18, 2022
@bqv
Copy link

bqv commented Apr 8, 2024

Morgan Freeman: But no further investigation occured.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-Major Severely degrades major functionality or product features, with no satisfactory workaround T-Defect Bugs, crashes, hangs, vulnerabilities, or other reported problems.
Projects
None yet
Development

No branches or pull requests

3 participants