From d5715e60f7b9c0669423bfe9b23fab6199e7c5dc Mon Sep 17 00:00:00 2001
From: Art Cancro <ajc@citadel.org>
Date: Tue, 27 Nov 2007 22:40:47 +0000
Subject: [PATCH] When restarting the XMPP stream after a successful SASL
 authentication, don't offer SASL as a stream feature again.  This causes the
 client to attempt re-authentication and/or go into an error condition.

---
 citadel/modules/jabber/serv_xmpp.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/citadel/modules/jabber/serv_xmpp.c b/citadel/modules/jabber/serv_xmpp.c
index a2a0c3643..7c19e509d 100644
--- a/citadel/modules/jabber/serv_xmpp.c
+++ b/citadel/modules/jabber/serv_xmpp.c
@@ -85,7 +85,9 @@ void xmpp_stream_start(void *data, const char *supplied_el, const char **attr)
 	cprintf("<session xmlns=\"urn:ietf:params:xml:ns:xmpp-session\"/>");
 
 	/* A really bad SASL implementation... */
-	xmpp_output_auth_mechs();
+	if (!CC->logged_in) {
+		xmpp_output_auth_mechs();
+	}
 
 	/* ...and the ability to close XML tags using angle brackets.  We should patent this. */
 	cprintf("</stream:features>");
-- 
2.39.2