XMPP неверный адрес отправителя
В сообщениях всем активным клиентам одного пользователя неправильно написан адрес отправителя.
Это широковещательная функция:
private void sendAllSessions(final StringBuilder message, final Entity sender,
final ServerRuntimeContext serverContext, EMailAddress to) {
EntityImpl recipient = new EntityImpl(to.getLocalName().getLocalName(), to.getDomain().getDomainName(), null);
Stanza build = createStanza(message, sender, recipient);
for (SessionContext sessionContext : serverContext.getResourceRegistry().getSessions(recipient)) {
SessionState state = sessionContext.getState();
SessionStateHolder stateHolder = new SessionStateHolder();
stateHolder.setState(state);
Stanza stanza = new MessageStanza(build);
LOG.severe("Send xmpp stanza: " + stanza + " from " + stanza.getFrom());
serverContext.getStanzaProcessor().processStanza(serverContext, sessionContext, stanza, stateHolder);
}
}
private Stanza createStanza(final StringBuilder strb, final Entity sender, final EntityImpl recipient) {
try {
LOG.severe("Create xmpp stanza from " + sender + " (" + sender.getFullQualifiedName() + ") to recipient "
+ recipient + " (" + recipient.getFullQualifiedName() + ")!");
StanzaBuilder sb = StanzaBuilder.createMessageStanza(sender, recipient, MessageStanzaType.HEADLINE, "html",
strb.toString());
Stanza build = sb.build();
return build;
} catch (RuntimeException e) {
StanzaBuilder sb = StanzaBuilder.createMessageStanza(sender, recipient, MessageStanzaType.NORMAL, null,
strb.toString());
Stanza build = sb.build();
return build;
}
}
Это ведение журнала:
07-Oct-2021 12:43:58.220 SEVERE [http-nio-80-exec-147] xx.DefaultChat.createStanza Create xmpp stanza from release-ma@example.com (release-ma@example.com) to recipient admin@example.com (admin@example.com)!
07-Oct-2021 12:43:58.222 SEVERE [http-nio-80-exec-147] xx.DefaultChat.sendAllSessions Send xmpp stanza: message.body.Thanks, please wait... from release-ma@example.com
07-Oct-2021 12:43:58.229 FINE [NioProcessor-2] org.apache.vysper.mina.StanzaLoggingFilter.messageSent ><message xmlns="jabber:client" to="release-ma@example.com" from="example.com" type="error"><body>Thanks, please wait...</body><error type="modify"><unknown-sender xmlns="urn:ietf:params:xml:ns:xmpp-stanzas"></unknown-sender></error></message>
Человеко-читаемый xml:
<message xmlns="jabber:client" to="release-ma@example.com"
from="example.com" type="error">
<body>Thanks, please wait...</body>
<error type="modify">
<unknown-sender
xmlns="urn:ietf:params:xml:ns:xmpp-stanzas"></unknown-sender>
</error>
</message>
У меня вопрос: почему в XML-Stanza указывается, что отправитель (из поля) является только доменом?
2 ответа
Это ответ сервера с ошибкой, поэтому это сообщение отправляет сервер, а не конкретный пользователь.
Если посмотреть исходный код обработки ошибки:
StanzaBuilder responseBuilder = StanzaBuilder.createDirectReply(stanza, true, "error");
Второй параметр - это параметр fromIsServerOnly:
public static StanzaBuilder createDirectReply(XMPPCoreStanza original, boolean fromIsServerOnly, String type)
И в этом случае созданная Станца удаляет адрес отправителя только как домен:
if (fromIsServerOnly)
newFrom = new EntityImpl(null, newFrom.getDomain(), null);
stanzaBuilder.addAttribute("from", newFrom.getFullQualifiedName());
После некоторой отладки я заметил эти строки (в org.apache.vysper.xmpp.protocol.ProtocolWorker):
// make sure that 'from' (if present) matches the bare authorized entity
// else respond with a stanza error 'unknown-sender'
// see rfc3920_draft-saintandre-rfc3920bis-04.txt#8.5.4
if (from != null && sessionContext.getInitiatingEntity() != null) {
Entity fromBare = from.getBareJID();
Entity initiatingEntity = sessionContext.getInitiatingEntity();
if (!initiatingEntity.equals(fromBare)) {
responseWriter.handleWrongFromJID(sessionContext, stanza);
return;
}
}
Поэтому я добавил строку:
private void sendAllSessions(final StringBuilder message, final Entity sender,
final ServerRuntimeContext serverContext, final EMailAddress recipientAddress) {
EntityImpl recipient = new EntityImpl(recipientAddress.getLocalName().getLocalName(),
recipientAddress.getDomain().getDomainName(), null);
Stanza build = createStanza(message, sender, recipient);
for (SessionContext sessionContext : serverContext.getResourceRegistry().getSessions(recipient)) {
SessionState state = sessionContext.getState();
SessionStateHolder stateHolder = new SessionStateHolder();
stateHolder.setState(state);
Stanza stanza = new MessageStanza(build);
LOG.severe("Send xmpp stanza: " + stanza + " from " + stanza.getFrom());
sessionContext.setInitiatingEntity(sender); // THIS LINE IS ADDED
serverContext.getStanzaProcessor().processStanza(serverContext, sessionContext, stanza, stateHolder);
}
}
Теперь он отлично работает:
<message type="headline" from="xxx" to="xxx" xml:lang="html">
<body>Test</body>
</message>