Opened 9 years ago

Closed 9 years ago

#12331 closed defect (fixed)

XMPP plugin in Pidgin 2.7.1 does not accept streams without 'id' attribute anymore

Reported by: liangent Owned by: deryni
Milestone: 2.7.4 Component: XMPP
Version: 2.7.1 Keywords:
Cc:

Description

I'm upgrading from 2.6.5 to 2.7.1 and in 2.7.1 I receive a message saying "XMPP stream missing ID" and an XMPP account cannot connect anymore (it works in 2.6.5). In RFC 3920, 'id' attribute is only 'SHOULD', not 'MUST'.

Attachments (1)

xmpp.log (1.7 KB) - added by liangent 9 years ago.
Related log. Some data is edited for privacy (JID stuff etc).

Download all attachments as: .zip

Change History (11)

comment:1 follow-up: Changed 9 years ago by darkrain42

  • Status changed from new to pending

Out of curiosity, what server doesn't send a stream ID? (The 'id' attribute is a MUST in the draft replacement for 3920)

comment:2 in reply to: ↑ 1 Changed 9 years ago by liangent

  • Status changed from pending to new

Replying to darkrain42:

Out of curiosity, what server doesn't send a stream ID? (The 'id' attribute is a MUST in the draft replacement for 3920)

talk.xiaonei.com

comment:3 Changed 9 years ago by darkrain42

  • Status changed from new to pending

Please follow the instructions to get a debug log and attach it to this ticket.
I'll figure out how to make the change, but I'm curious what software is running behind this server

Changed 9 years ago by liangent

Related log. Some data is edited for privacy (JID stuff etc).

comment:4 Changed 9 years ago by liangent

  • Status changed from pending to new

Attachment (xmpp.log) added by ticket reporter.

comment:6 follow-up: Changed 9 years ago by darkrain42

Oops, I guess that test needs to be run with a working account (I'm interested in the line that matches on 'http://jabber.org/protocol/disco#info').

comment:7 in reply to: ↑ 6 Changed 9 years ago by liangent

Replying to darkrain42:

Oops, I guess that test needs to be run with a working account (I'm interested in the line that matches on 'http://jabber.org/protocol/disco#info').

What did you mean and do I need to do something more?

comment:8 Changed 9 years ago by darkrain42

Well, if you could get a debug log from 2.6.5, that'd be great, but if not, don't worry about it.

comment:9 Changed 9 years ago by liangent

If I don't downgrade Pidgin, is there a way to change the behavior back, for example, copying some files taken from an older version to ~/.purple/plugins ?

comment:10 Changed 9 years ago by darkrain42

  • Milestone set to 2.7.4

Grrr, dropped this on the floor. Sorry.

comment:11 Changed 9 years ago by darkrain42@…

  • Resolution set to fixed
  • Status changed from new to closed

(In fc48e1385d53f406de694f252adc2fd368ba1277):
jabber: Make the reliance on stream ID laxer. Closes #12331

The requirement that there be an ID on the responder's header was underspecified/just a SHOULD (depending on how you read it) in rfc3920.

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!