From a4db3f70468c31c0d579e6b9dcd4e90cd9f0d1a8 Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Thu, 21 Sep 2017 08:21:43 +0200 Subject: tests: make the imap server not verify user+password ... as the test cases themselves do that and it makes it easier to add crazy test cases. Test 800 updated to use user name + password that need quoting. Test 856 updated to trigger an auth fail differently. Ref: #1902 --- tests/data/test800 | 4 ++-- tests/data/test856 | 3 +++ tests/ftpserver.pl | 15 +++------------ 3 files changed, 8 insertions(+), 14 deletions(-) diff --git a/tests/data/test800 b/tests/data/test800 index 6b29f7a2b..1ac1da59b 100644 --- a/tests/data/test800 +++ b/tests/data/test800 @@ -31,7 +31,7 @@ imap IMAP FETCH message -'imap://%HOSTIP:%IMAPPORT/800/;UID=1' -u user:secret +'imap://%HOSTIP:%IMAPPORT/800/;UID=1' -u "user*:secret{" @@ -40,7 +40,7 @@ IMAP FETCH message A001 CAPABILITY -A002 LOGIN user secret +A002 LOGIN "user*" "secret{" A003 SELECT 800 A004 FETCH 1 BODY[] A005 LOGOUT diff --git a/tests/data/test856 b/tests/data/test856 index 4d8d7ed61..dcb07c5a9 100644 --- a/tests/data/test856 +++ b/tests/data/test856 @@ -10,6 +10,9 @@ FAILURE # # Server-side + +REPLY PASS -ERR Login failure + # diff --git a/tests/ftpserver.pl b/tests/ftpserver.pl index 081298f1b..2aba4263a 100755 --- a/tests/ftpserver.pl +++ b/tests/ftpserver.pl @@ -6,7 +6,7 @@ # | (__| |_| | _ <| |___ # \___|\___/|_| \_\_____| # -# Copyright (C) 1998 - 2014, Daniel Stenberg, , et al. +# Copyright (C) 1998 - 2014, 2017, Daniel Stenberg, , et al. # # This software is licensed as described in the file COPYING, which # you should have received as part of this distribution. The terms @@ -174,7 +174,6 @@ my $exit_signal; # first signal handled in exit_signal_handler #********************************************************************** # Mail related definitions # -my $TEXT_USERNAME = "user"; my $TEXT_PASSWORD = "secret"; my $POP3_TIMESTAMP = "<1972.987654321\@curl>"; @@ -1121,9 +1120,6 @@ sub LOGIN_imap { if ($user eq "") { sendcontrol "$cmdid BAD Command Argument\r\n"; } - elsif (($user ne $TEXT_USERNAME) || ($password ne $TEXT_PASSWORD)) { - sendcontrol "$cmdid NO LOGIN failed\r\n"; - } else { sendcontrol "$cmdid OK LOGIN completed\r\n"; } @@ -1681,7 +1677,7 @@ sub APOP_pop3 { else { my $digest = Digest::MD5::md5_hex($POP3_TIMESTAMP, $TEXT_PASSWORD); - if (($user ne $TEXT_USERNAME) || ($secret ne $digest)) { + if ($secret ne $digest) { sendcontrol "-ERR Login failure\r\n"; } else { @@ -1740,12 +1736,7 @@ sub PASS_pop3 { logmsg "PASS_pop3 got $password\n"; - if (($username ne $TEXT_USERNAME) || ($password ne $TEXT_PASSWORD)) { - sendcontrol "-ERR Login failure\r\n"; - } - else { - sendcontrol "+OK Login successful\r\n"; - } + sendcontrol "+OK Login successful\r\n"; return 0; } -- cgit v1.2.1