[Pidgin] #5565: Should XMPP resource default to the local machine's hostname?
Pidgin
trac at pidgin.im
Mon Jun 16 16:22:26 EDT 2008
#5565: Should XMPP resource default to the local machine's hostname?
---------------------+------------------------------------------------------
Reporter: resiak | Owner: deryni
Type: patch | Status: assigned
Priority: minor | Milestone:
Component: XMPP | Version: 2.4.1
Resolution: | Keywords:
Pending: 0 |
---------------------+------------------------------------------------------
Changes (by deryni):
* status: new => assigned
Comment:
Support for resource binding is a MUST in the new proposed text of RFC
3920 (and was basically always a MUST for clients that wanted to function
correctly in general) so I feel comfortable assuming that all servers
support it.
As to the patch, using an explicit __NONE__ seems like a poor idea to me
when we could just as easily use an empty or all-whitespace resource for
that purpose (and is almost entirely self-documenting). pidgin has a
purple_strreplace function already so there is no need to create a new one
just for jabber. (Also, on purely nit-picking points, pidgin does not
allow // style comments.) Other than that the patch looks fine to me, so
you can either clean those things up yourself or let me do it when I get
to applying the patch.
The whole point of using Home, as I explained before, is that it was
*supposed* to be clearly wrong sometimes and thus spark people to realize
that they can change it to say whatever they want it to say.
Support for random numeric resources would be nice, but if that was going
to happen it would be best for them to be GUIDs at which point we might as
well leave that up to the server to handle for us (by not sending a
resource at all).
--
Ticket URL: <http://developer.pidgin.im/ticket/5565#comment:10>
Pidgin <http://pidgin.im>
Pidgin
More information about the Tracker
mailing list