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)