From acb207edba53df542f08219b2f241282b2e75279 Mon Sep 17 00:00:00 2001 From: Jeffrey Stedfast Date: Thu, 6 Nov 2003 16:26:26 +0000 Subject: Don't bother trying to see if the server advertises EHLO. Simply always 2003-11-05 Jeffrey Stedfast * providers/smtp/camel-smtp-transport.c (connect_to_server): Don't bother trying to see if the server advertises EHLO. Simply always try EHLO and fall back to HELO if EHLO fails. Fixes bug #50535. svn path=/trunk/; revision=23199 --- camel/providers/smtp/camel-smtp-transport.c | 27 +++++++++------------------ 1 file changed, 9 insertions(+), 18 deletions(-) (limited to 'camel/providers/smtp') diff --git a/camel/providers/smtp/camel-smtp-transport.c b/camel/providers/smtp/camel-smtp-transport.c index 6e15b33ce6..b8dbaa6c1c 100644 --- a/camel/providers/smtp/camel-smtp-transport.c +++ b/camel/providers/smtp/camel-smtp-transport.c @@ -309,27 +309,18 @@ connect_to_server (CamelService *service, int try_starttls, CamelException *ex) g_free (respbuf); return FALSE; } - if (strstr (respbuf, "ESMTP")) - transport->flags |= CAMEL_SMTP_TRANSPORT_IS_ESMTP; } while (*(respbuf+3) == '-'); /* if we got "220-" then loop again */ g_free (respbuf); - /* send EHLO (or HELO, depending on the service type) */ - if (!(transport->flags & CAMEL_SMTP_TRANSPORT_IS_ESMTP)) { - /* If we did not auto-detect ESMTP, we should still send EHLO */ - transport->flags |= CAMEL_SMTP_TRANSPORT_IS_ESMTP; - if (!smtp_helo (transport, ex)) { - if (!transport->connected) - return FALSE; - - /* Okay, apparently this server doesn't support ESMTP */ - camel_exception_clear (ex); - transport->flags &= ~CAMEL_SMTP_TRANSPORT_IS_ESMTP; - if (!smtp_helo (transport, ex) && !transport->connected) - return FALSE; - } - } else { - /* send EHLO */ + /* Try sending EHLO */ + transport->flags |= CAMEL_SMTP_TRANSPORT_IS_ESMTP; + if (!smtp_helo (transport, ex)) { + if (!transport->connected) + return FALSE; + + /* Fall back to HELO */ + camel_exception_clear (ex); + transport->flags &= ~CAMEL_SMTP_TRANSPORT_IS_ESMTP; if (!smtp_helo (transport, ex) && !transport->connected) return FALSE; } -- cgit v1.2.3