[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[gnurl] 158/282: smtp: Support UTF-8 based host names in the VRFY comman
From: |
gnunet |
Subject: |
[gnurl] 158/282: smtp: Support UTF-8 based host names in the VRFY command |
Date: |
Wed, 01 Apr 2020 14:30:23 +0200 |
This is an automated email from the git hooks/post-receive script.
ng0 pushed a commit to branch master
in repository gnurl.
commit 2aa6c1735a84f6df11b5f5863a704034b820c925
Author: Steve Holme <address@hidden>
AuthorDate: Sat Feb 8 00:37:03 2020 +0000
smtp: Support UTF-8 based host names in the VRFY command
---
lib/smtp.c | 34 ++++++++++++++++++++++++++++------
tests/data/Makefile.inc | 2 +-
tests/data/test961 | 3 +++
tests/data/{test961 => test964} | 16 ++++++++--------
tests/ftpserver.pl | 8 +++++++-
5 files changed, 47 insertions(+), 16 deletions(-)
diff --git a/lib/smtp.c b/lib/smtp.c
index 1718cc2db..3c3dfcbd9 100644
--- a/lib/smtp.c
+++ b/lib/smtp.c
@@ -485,13 +485,35 @@ static CURLcode smtp_perform_command(struct connectdata
*conn)
struct Curl_easy *data = conn->data;
struct SMTP *smtp = data->req.protop;
- /* Send the command */
- if(smtp->rcpt)
- result = Curl_pp_sendf(&conn->proto.smtpc.pp, "%s %s",
- smtp->custom && smtp->custom[0] != '\0' ?
- smtp->custom : "VRFY",
- smtp->rcpt->data);
+ if(smtp->rcpt) {
+ if((!smtp->custom) || (!smtp->custom[0])) {
+ char *address = NULL;
+ struct hostname host = { NULL, NULL, NULL, NULL };
+
+ /* Parse the mailbox to verify into the local address and host name
+ parts, converting the host name to an IDN A-label if necessary */
+ result = smtp_parse_address(conn, smtp->rcpt->data,
+ &address, &host);
+ if(result)
+ return result;
+
+ /* Send the VRFY command (Note: The host name part may be absent when the
+ host is a local system) */
+ result = Curl_pp_sendf(&conn->proto.smtpc.pp, "VRFY %s%s%s",
+ address,
+ host.name ? "@" : "",
+ host.name ? host.name : "");
+
+ Curl_free_idnconverted_hostname(&host);
+ free(address);
+ }
+ else
+ /* Send the custom recipient based command such as the EXPN command */
+ result = Curl_pp_sendf(&conn->proto.smtpc.pp, "%s %s", smtp->custom,
+ smtp->rcpt->data);
+ }
else
+ /* Send the non-recipient based command such as HELP */
result = Curl_pp_sendf(&conn->proto.smtpc.pp, "%s",
smtp->custom && smtp->custom[0] != '\0' ?
smtp->custom : "HELP");
diff --git a/tests/data/Makefile.inc b/tests/data/Makefile.inc
index 8d2ef14ce..5215f442f 100644
--- a/tests/data/Makefile.inc
+++ b/tests/data/Makefile.inc
@@ -109,7 +109,7 @@ test927 test928 test929 test930 test931 test932 test933
test934 test935 \
test936 test937 test938 test939 test940 test941 test942 test943 test944 \
test945 test946 test947 test948 test949 test950 test951 test952 test953 \
test954 test955 test956 test957 test958 test959 test960 test961 test962 \
-test963 \
+test963 test964 \
\
test1000 test1001 test1002 test1003 test1004 test1005 test1006 test1007 \
test1008 test1009 test1010 test1011 test1012 test1013 test1014 test1015 \
diff --git a/tests/data/test961 b/tests/data/test961
index 80ebca887..2e70cb404 100644
--- a/tests/data/test961
+++ b/tests/data/test961
@@ -17,6 +17,9 @@ VRFY
<server>
smtp
</server>
+<features>
+!idn
+</features>
<setenv>
LC_ALL=en_US.UTF-8
LC_CTYPE=en_US.UTF-8
diff --git a/tests/data/test961 b/tests/data/test964
similarity index 68%
copy from tests/data/test961
copy to tests/data/test964
index 80ebca887..326cdd21e 100644
--- a/tests/data/test961
+++ b/tests/data/test964
@@ -3,6 +3,7 @@
<keywords>
SMTP
VRFY
+IDN
</keywords>
</info>
@@ -17,6 +18,9 @@ VRFY
<server>
smtp
</server>
+<features>
+idn
+</features>
<setenv>
LC_ALL=en_US.UTF-8
LC_CTYPE=en_US.UTF-8
@@ -25,23 +29,19 @@ LC_CTYPE=en_US.UTF-8
perl -MI18N::Langinfo=langinfo,CODESET -e 'die "Needs a UTF-8 locale" if
(lc(langinfo(CODESET())) ne "utf-8");'
</precheck>
<name>
-SMTP external VRFY without SMTPUTF8 support - UTF-8 based recipient (host part
only)
+SMTP external VRFY without SMTPUTF8 support (IDN Enabled) - UTF-8 based
recipient (host part only)
</name>
<command>
-smtp://%HOSTIP:%SMTPPORT/961 --mail-rcpt user@åäö.se
+smtp://%HOSTIP:%SMTPPORT/964 --mail-rcpt user@åäö.se
</command>
</client>
#
# Verify data after the test has been "shot"
<verify>
-# 56 - CURLE_RECV_ERROR
-<errorcode>
-56
-</errorcode>
<protocol>
-EHLO 961
-VRFY user@åäö.se
+EHLO 964
+VRFY address@hidden
QUIT
</protocol>
</verify>
diff --git a/tests/ftpserver.pl b/tests/ftpserver.pl
index 6709769d7..59a1665bd 100755
--- a/tests/ftpserver.pl
+++ b/tests/ftpserver.pl
@@ -1040,7 +1040,13 @@ sub VRFY_smtp {
my @data = getreplydata($smtp_client);
if(!@data) {
- push @data, "250 <$username\@example.com>\r\n"
+ if ($username !~
+
/^([a-zA-Z0-9._%+-]+)\@(([a-zA-Z0-9-]+)\.)+([a-zA-Z]{2,4})$/) {
+ push @data, "250 <$username\@example.com>\r\n"
+ }
+ else {
+ push @data, "250 <$username>\r\n"
+ }
}
for my $d (@data) {
--
To stop receiving notification emails like this one, please contact
address@hidden.
- [gnurl] 147/282: test1610: Fixed the link to the unit test, (continued)
- [gnurl] 147/282: test1610: Fixed the link to the unit test, gnunet, 2020/04/01
- [gnurl] 150/282: ntlm: Moved the HMAC MD5 function into the HMAC module as a generic function, gnunet, 2020/04/01
- [gnurl] 140/282: http2: now require nghttp2 >= 1.12.0, gnunet, 2020/04/01
- [gnurl] 123/282: socks: make the connect phase non-blocking, gnunet, 2020/04/01
- [gnurl] 145/282: md5/sha256: Updated the functions to allow non-string data to be hashed, gnunet, 2020/04/01
- [gnurl] 153/282: ftpserver: Updated VRFY_smtp() so the response isn't necessary in the test case, gnunet, 2020/04/01
- [gnurl] 148/282: md4: Use const for the length input parameter, gnunet, 2020/04/01
- [gnurl] 152/282: ftpserver: Corrected the e-mail address regex in MAIL_smtp() and RCTP_smtp(), gnunet, 2020/04/01
- [gnurl] 155/282: url: Make the IDN conversion functions available to others, gnunet, 2020/04/01
- [gnurl] 159/282: smtp: Detect server support for the UTF-8 extension as defined in RFC-6531, gnunet, 2020/04/01
- [gnurl] 158/282: smtp: Support UTF-8 based host names in the VRFY command,
gnunet <=
- [gnurl] 151/282: hmac: Added a unit test for the HMAC hash generation, gnunet, 2020/04/01
- [gnurl] 149/282: tests: Added a unit test for MD4 digest generation, gnunet, 2020/04/01
- [gnurl] 163/282: smtp: Support the SMTPUTF8 extension for the EXPN command, gnunet, 2020/04/01
- [gnurl] 162/282: smtp: Support the SMTPUTF8 extension in the VRFY command, gnunet, 2020/04/01
- [gnurl] 164/282: smtp: Tidy up, following recent changes, to maintain the coding style, gnunet, 2020/04/01
- [gnurl] 156/282: smtp: Support UTF-8 based host names in the MAIL command, gnunet, 2020/04/01
- [gnurl] 161/282: smtp: Support the SMTPUTF8 extension in the RCPT TO command, gnunet, 2020/04/01
- [gnurl] 157/282: smtp: Support UTF-8 based host names in the RCPT TO command, gnunet, 2020/04/01
- [gnurl] 165/282: http: added 417 response treatment, gnunet, 2020/04/01
- [gnurl] 166/282: cookie: remove unnecessary check for 'out != 0', gnunet, 2020/04/01