Opened 9 years ago

Last modified 9 years ago

#12016 new defect

XMPP message error stanzas only displayed if they have a <body/>

Reported by: Kev (from XMPP) Owned by: deryni
Milestone: Component: XMPP
Version: 2.7.0 Keywords:
Cc:

Description (last modified by darkrain42)

XMPP message errors are only displayed if they have a <body/>

static void handle_error(JabberMessage *jm)
{
	char *buf;

	if(!jm->body)
		return;
...

Someone reported to Kev that Pidgin wasn't displaying message errors without the <body/>. My guess is that this is a by-product of the day when all errors had the full stanza contents echoed back along with an <error/> sub-element.

Change History (2)

comment:1 Changed 9 years ago by darkrain42

Hrm, I think I've seen PEP deliveries produce <message type='error'/> responses (ejabberd's offline storage was full or was delivering to GTalk-being-buggy). This needs testing that it doesn't produce too many false positives (or check what other clients do).

comment:2 Changed 9 years ago by darkrain42

  • Description modified (diff)
Note: See TracTickets for help on using tickets.
All information, including names and email addresses, entered onto this website or sent to mailing lists affiliated with this website will be public. Do not post confidential information, especially passwords!