Mercurial > pidgin.yaz
comparison libpurple/protocols/jabber/parser.c @ 30896:4072761e3454
jabber: Slight improvements to the stream ID fix
This warns that it will fail miserably with digestmd5 legacy auth.
Also, document that crammd5 code and remove a check for js->stream_id
since crammd5 doesn't use it (no salt!)
author | Paul Aurich <paul@darkrain42.org> |
---|---|
date | Tue, 31 Aug 2010 02:52:28 +0000 |
parents | ec12d2883266 |
children | 63ce2608c2c0 |
comparison
equal
deleted
inserted
replaced
30895:8084631e2e84 | 30896:4072761e3454 |
---|---|
100 */ | 100 */ |
101 purple_connection_error_reason(js->gc, | 101 purple_connection_error_reason(js->gc, |
102 PURPLE_CONNECTION_ERROR_AUTHENTICATION_IMPOSSIBLE, | 102 PURPLE_CONNECTION_ERROR_AUTHENTICATION_IMPOSSIBLE, |
103 _("XMPP stream missing ID")); | 103 _("XMPP stream missing ID")); |
104 #else | 104 #else |
105 /* Instead, let's make up a fancy-schmancy stream ID, which | 105 /* Instead, let's make up a placeholder stream ID, which we need |
106 * we need to do because we flag on js->stream_id == NULL being | 106 * to do because we flag on it being NULL as a special case |
107 * a special case in this function. | 107 * in this parsing code. |
108 */ | 108 */ |
109 js->stream_id = purple_uuid_random(); | 109 js->stream_id = g_strdup(""); |
110 purple_debug_info("jabber", "Server failed to specify a stream " | |
111 "ID (underspecified in rfc3920, but intended " | |
112 "to be a MUST; digest legacy auth may fail."); | |
110 #endif | 113 #endif |
111 } | 114 } |
112 } else { | 115 } else { |
113 | 116 |
114 if(js->current) | 117 if(js->current) |