[#LM-49] loudmouth does not determine server_from_jid correctly

Loudmouth

loudmouth does not determine server_from_jid correctly

Created: 06/Aug/06 02:20 PM Updated: 24/Feb/07 04:08 AM

Component/s:

None

Affects Version/s:

1.1.2

Fix Version/s:

1.3.1, 1.2.1

File Attachments:

loudmouth-server_from_jid.diff (0.9 kb)

Description

« Hide

In lm-connection.c, the domain is extracted from the jid if it contains a '@'. Everything after that is considered the domain name. If the jid contains a '/'+resource, the resource becomes part of the domain instead of being stripped off, thus preventing successful connections

All

Comments

Change History

Sort Order:

[ Permlink | « Hide ]

Matthias Quasthoff - [06/Aug/06 02:21 PM ]

this patch seems to fix this issue

[ Permlink | « Hide ]

Mikael Hallendal - [10/Sep/06 11:16 PM ]

Marking for 1.2

[ Permlink | « Hide ]

Mikael Hallendal - [02/Feb/07 10:31 AM ]

Moving to 1.2.1 as I will not make any more changes before releasing 1.2.0

[ Permlink | « Hide ]

Mikael Hallendal - [24/Feb/07 04:08 AM ]

Commited this to both 1.2 and 1.3 branches.

Not entirely sure if we shouldn't add a check that makes sure the resource is not included when calling lm_connection_set_jid but even if we do this patch will not mess things up at least.

Sorry it took so long to get around to add it.

Thanks a lot!