From 9b72e1032026215cbce2daa0bce9bfe342eac58b Mon Sep 17 00:00:00 2001 From: Oswald Buddenhagen Date: Thu, 5 May 2022 11:33:08 +0200 Subject: [PATCH] make IMAP error messages less confusing don't print the status, as the user doesn't really care whether it's NO or BAD. more importantly, "NO LOGIN failed" is a rather misleading thing to report. --- src/drv_imap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/drv_imap.c b/src/drv_imap.c index dd5977a..7074e43 100644 --- a/src/drv_imap.c +++ b/src/drv_imap.c @@ -1904,13 +1904,13 @@ imap_socket_read( void *aux ) resp = RESP_CANCEL; } } - error( "IMAP command '%s' returned an error: %s %s\n", + error( "IMAP command '%s' returned an error: %s\n", starts_with( cmdp->cmd, -1, "LOGIN", 5 ) ? "LOGIN " : starts_with( cmdp->cmd, -1, "AUTHENTICATE PLAIN", 18 ) ? "AUTHENTICATE PLAIN " : cmdp->cmd, - arg, cmd ? cmd : "" ); + cmd ? cmd : "" ); } doresp: if ((resp2 = parse_response_code( ctx, cmdp, cmd )) > resp)