commit 9b72e1032026215cbce2daa0bce9bfe342eac58b
Author: Oswald Buddenhagen <o...@users.sf.net>
Date:   Thu May 5 11:33:08 2022 +0200

    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 dd5977a4..7074e437 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 <user> <pass>" :
                                           starts_with( cmdp->cmd, -1, 
"AUTHENTICATE PLAIN", 18 ) ?
                                               "AUTHENTICATE PLAIN <authdata>" :
                                                cmdp->cmd,
-                                      arg, cmd ? cmd : "" );
+                                      cmd ? cmd : "<unspecified>" );
                        }
                  doresp:
                        if ((resp2 = parse_response_code( ctx, cmdp, cmd )) > 
resp)


_______________________________________________
isync-devel mailing list
isync-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/isync-devel

Reply via email to