Commit 8baf255a authored by Tom Evans's avatar Tom Evans

OF-670: Detect all non-recoverable errors

Kick MUC user when a corresponding packet is returned with any permanent
error condition (PacketError.Type.cancel).
parent 753b9584
...@@ -309,10 +309,10 @@ public class MultiUserChatServiceImpl implements Component, MultiUserChatService ...@@ -309,10 +309,10 @@ public class MultiUserChatServiceImpl implements Component, MultiUserChatService
return; return;
} }
} }
// OF-670: Kick S2S users for undeliverable message bounces; // OF-670: Kick MUC users who return permanent error conditions;
// error condition set via LocalOutgoingServerSession or OutgoingSessionPromise // also detects S2S-based users from non-responsive domains.
if (packet.getError() != null && if (packet.getError() != null &&
packet.getError().getType().equals(PacketError.Condition.remote_server_not_found)) { packet.getError().getType().equals(PacketError.Type.cancel)) {
removeUser(packet.getFrom()); removeUser(packet.getFrom());
} }
else { else {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment