260 lines
9.1 KiB
Diff
260 lines
9.1 KiB
Diff
From 2f34a7347f315513bfda9ef14770d287fb246bcd Mon Sep 17 00:00:00 2001
|
|
From: Daniel Stenberg <daniel@haxx.se>
|
|
Date: Thu, 1 Dec 2022 09:21:04 +0100
|
|
Subject: [PATCH] runtests: do CRLF replacements per section only
|
|
|
|
The `crlf="yes"` attribute and "hyper mode" are now only applied on a
|
|
subset of dedicated sections: data, datacheck, stdout and protocol.
|
|
|
|
Updated test 2500 accordingly.
|
|
|
|
Also made test1 use crlf="yes" for <protocol>, mostly because it is
|
|
often used as a template test case. Going forward, using this attribute
|
|
we should be able to write test cases using linefeeds only and avoid
|
|
mixed line ending encodings.
|
|
|
|
Follow-up to ca15b7512e8d11
|
|
|
|
Fixes #10009
|
|
Closes #10010
|
|
|
|
Conflict: remove tests/data/test2500
|
|
Reference: https://github.com/curl/curl/commit/2f34a7347f315513bfda9ef14770d287fb246bcd
|
|
---
|
|
tests/FILEFORMAT.md | 20 +++++++++++++-----
|
|
tests/data/test1 | 14 ++++++-------
|
|
tests/runtests.pl | 49 +++++++++++++++++++++++++++++++++++++++++----
|
|
3 files changed, 67 insertions(+), 16 deletions(-)
|
|
|
|
diff --git a/tests/FILEFORMAT.md b/tests/FILEFORMAT.md
|
|
index e17605b..c6e65a4 100644
|
|
--- a/tests/FILEFORMAT.md
|
|
+++ b/tests/FILEFORMAT.md
|
|
@@ -187,7 +187,7 @@ When using curl built with Hyper, the keywords must include HTTP or HTTPS for
|
|
'hyper mode' to kick in and make line ending checks work for tests.
|
|
## `<reply>`
|
|
|
|
-### `<data [nocheck="yes"] [sendzero="yes"] [base64="yes"] [hex="yes"]>`
|
|
+### `<data [nocheck="yes"] [sendzero="yes"] [base64="yes"] [hex="yes"] [crlf="yes"]>`
|
|
|
|
data to be sent to the client on its request and later verified that it
|
|
arrived safely. Set `nocheck="yes"` to prevent the test script from verifying
|
|
@@ -213,12 +213,16 @@ much sense for other sections than "data").
|
|
`hex=yes` means that the data is a sequence of hex pairs. It will get decoded
|
|
and used as "raw" data.
|
|
|
|
+`crlf=yes` forces *header* newlines to become CRLF even if not written so in
|
|
+the source file. Note that this makes runtests.pl parse and "guess" what is a
|
|
+header and what is not in order to apply the CRLF line endings appropriately.
|
|
+
|
|
For FTP file listings, the `<data>` section will be used *only* if you make
|
|
sure that there has been a CWD done first to a directory named `test-[num]`
|
|
where [num] is the test case number. Otherwise the ftp server can't know from
|
|
which test file to load the list content.
|
|
|
|
-### `<dataNUM>`
|
|
+### `<dataNUM [crlf="yes"]>`
|
|
|
|
Send back this contents instead of the <data> one. The num is set by:
|
|
|
|
@@ -242,7 +246,7 @@ The connect section is used instead of the 'data' for all CONNECT
|
|
requests. The remainder of the rules for the data section then apply but with
|
|
a connect prefix.
|
|
|
|
-### `<datacheck [mode="text"] [nonewline="yes"]>`
|
|
+### `<datacheck [mode="text"] [nonewline="yes"] [crlf="yes"]>`
|
|
if the data is sent but this is what should be checked afterwards. If
|
|
`nonewline=yes` is set, runtests will cut off the trailing newline from the
|
|
data before comparing with the one actually received by the client.
|
|
@@ -250,7 +254,7 @@ data before comparing with the one actually received by the client.
|
|
Use the `mode="text"` attribute if the output is in text mode on platforms
|
|
that have a text/binary difference.
|
|
|
|
-### `<datacheckNUM [nonewline="yes"] [mode="text"]>`
|
|
+### `<datacheckNUM [nonewline="yes"] [mode="text"] [crlf="yes"]>`
|
|
The contents of numbered datacheck sections are appended to the non-numbered
|
|
one.
|
|
|
|
@@ -534,6 +538,9 @@ the trailing newline of this given data before comparing with the one actually
|
|
sent by the client The `<strip>` and `<strippart>` rules are applied before
|
|
comparisons are made.
|
|
|
|
+`crlf=yes` forces the newlines to become CRLF even if not written so in the
|
|
+test.
|
|
+
|
|
### `<proxy [nonewline="yes"][crlf="yes"]>`
|
|
|
|
The protocol dump curl should transmit to a HTTP proxy (when the http-proxy
|
|
@@ -550,7 +557,7 @@ have a text/binary difference.
|
|
If 'nonewline' is set, we will cut off the trailing newline of this given data
|
|
before comparing with the one actually received by the client
|
|
|
|
-### `<stdout [mode="text"] [nonewline="yes"]>`
|
|
+### `<stdout [mode="text"] [nonewline="yes"] [crlf="yes"]>`
|
|
This verifies that this data was passed to stdout.
|
|
|
|
Use the mode="text" attribute if the output is in text mode on platforms that
|
|
@@ -559,6 +566,9 @@ have a text/binary difference.
|
|
If 'nonewline' is set, we will cut off the trailing newline of this given data
|
|
before comparing with the one actually received by the client
|
|
|
|
+`crlf=yes` forces the newlines to become CRLF even if not written so in the
|
|
+test.
|
|
+
|
|
### `<file name="log/filename" [mode="text"]>`
|
|
The file's contents must be identical to this after the test is complete. Use
|
|
the mode="text" attribute if the output is in text mode on platforms that have
|
|
diff --git a/tests/data/test1 b/tests/data/test1
|
|
index f39a08b..700bed8 100644
|
|
--- a/tests/data/test1
|
|
+++ b/tests/data/test1
|
|
@@ -9,7 +9,7 @@ HTTP GET
|
|
#
|
|
# Server-side
|
|
<reply>
|
|
-<data>
|
|
+<data crlf="yes">
|
|
HTTP/1.1 200 OK
|
|
Date: Tue, 09 Nov 2010 14:49:00 GMT
|
|
Server: test-server/fake
|
|
@@ -42,12 +42,12 @@ http://%HOSTIP:%HTTPPORT/%TESTNUMBER
|
|
#
|
|
# Verify data after the test has been "shot"
|
|
<verify>
|
|
-<protocol>
|
|
-GET /%TESTNUMBER HTTP/1.1
|
|
-Host: %HOSTIP:%HTTPPORT
|
|
-User-Agent: curl/%VERSION
|
|
-Accept: */*
|
|
-
|
|
+<protocol crlf="yes">
|
|
+GET /%TESTNUMBER HTTP/1.1
|
|
+Host: %HOSTIP:%HTTPPORT
|
|
+User-Agent: curl/%VERSION
|
|
+Accept: */*
|
|
+
|
|
</protocol>
|
|
</verify>
|
|
</testcase>
|
|
diff --git a/tests/runtests.pl b/tests/runtests.pl
|
|
index 098b19e..fa6bcac 100755
|
|
--- a/tests/runtests.pl
|
|
+++ b/tests/runtests.pl
|
|
@@ -3419,7 +3419,13 @@ sub subBase64 {
|
|
|
|
my $prevupdate;
|
|
sub subNewlines {
|
|
- my ($thing) = @_;
|
|
+ my ($force, $thing) = @_;
|
|
+
|
|
+ if($force) {
|
|
+ # enforce CRLF newline
|
|
+ $$thing =~ s/\x0d*\x0a/\x0d\x0a/;
|
|
+ return;
|
|
+ }
|
|
|
|
# When curl is built with Hyper, it gets all response headers delivered as
|
|
# name/value pairs and curl "invents" the newlines when it saves the
|
|
@@ -3433,7 +3439,7 @@ sub subNewlines {
|
|
# skip curl error messages
|
|
($$thing !~ /^curl: \(\d+\) /))) {
|
|
# enforce CRLF newline
|
|
- $$thing =~ s/\x0a/\x0d\x0a/;
|
|
+ $$thing =~ s/\x0d*\x0a/\x0d\x0a/;
|
|
$prevupdate = 1;
|
|
}
|
|
else {
|
|
@@ -3505,6 +3511,7 @@ sub prepro {
|
|
my (@entiretest) = @_;
|
|
my $show = 1;
|
|
my @out;
|
|
+ my $data_crlf;
|
|
for my $s (@entiretest) {
|
|
my $f = $s;
|
|
if($s =~ /^ *%if (.*)/) {
|
|
@@ -3528,10 +3535,19 @@ sub prepro {
|
|
next;
|
|
}
|
|
if($show) {
|
|
+ # The processor does CRLF replacements in the <data*> sections if
|
|
+ # necessary since those parts might be read by separate servers.
|
|
+ if($s =~ /^ *<data(.*)\>/) {
|
|
+ if($1 =~ /crlf="yes"/ || $has_hyper) {
|
|
+ $data_crlf = 1;
|
|
+ }
|
|
+ }
|
|
+ elsif(($s =~ /^ *<\/data/) && $data_crlf) {
|
|
+ $data_crlf = 0;
|
|
+ }
|
|
subVariables(\$s, $testnum, "%");
|
|
subBase64(\$s);
|
|
- subNewlines(\$s) if($has_hyper && ($keywords{"HTTP"} ||
|
|
- $keywords{"HTTPS"}));
|
|
+ subNewlines(0, \$s) if($data_crlf);
|
|
push @out, $s;
|
|
}
|
|
}
|
|
@@ -3845,6 +3861,11 @@ sub singletest {
|
|
# of the datacheck
|
|
chomp($replycheckpart[$#replycheckpart]);
|
|
}
|
|
+ if($replycheckpartattr{'crlf'} ||
|
|
+ ($has_hyper && ($keywords{"HTTP"}
|
|
+ || $keywords{"HTTPS"}))) {
|
|
+ map subNewlines(0, \$_), @replycheckpart;
|
|
+ }
|
|
push(@reply, @replycheckpart);
|
|
}
|
|
}
|
|
@@ -3859,6 +3880,11 @@ sub singletest {
|
|
map s/\r\n/\n/g, @reply;
|
|
map s/\n/\r\n/g, @reply;
|
|
}
|
|
+ if($replyattr{'crlf'} ||
|
|
+ ($has_hyper && ($keywords{"HTTP"}
|
|
+ || $keywords{"HTTPS"}))) {
|
|
+ map subNewlines(0, \$_), @reply;
|
|
+ }
|
|
}
|
|
|
|
# this is the valid protocol blurb curl should generate
|
|
@@ -4300,6 +4326,12 @@ sub singletest {
|
|
chomp($validstdout[$#validstdout]);
|
|
}
|
|
|
|
+ if($hash{'crlf'} ||
|
|
+ ($has_hyper && ($keywords{"HTTP"}
|
|
+ || $keywords{"HTTPS"}))) {
|
|
+ map subNewlines(0, \$_), @validstdout;
|
|
+ }
|
|
+
|
|
$res = compare($testnum, $testname, "stdout", \@actual, \@validstdout);
|
|
if($res) {
|
|
return $errorreturncode;
|
|
@@ -4394,6 +4426,10 @@ sub singletest {
|
|
}
|
|
}
|
|
|
|
+ if($hash{'crlf'}) {
|
|
+ map subNewlines(1, \$_), @protstrip;
|
|
+ }
|
|
+
|
|
if((!$out[0] || ($out[0] eq "")) && $protstrip[0]) {
|
|
logmsg "\n $testnum: protocol FAILED!\n".
|
|
" There was no content at all in the file $SERVERIN.\n".
|
|
@@ -4530,6 +4566,11 @@ sub singletest {
|
|
map s/\r\n/\n/g, @outfile;
|
|
map s/\n/\r\n/g, @outfile;
|
|
}
|
|
+ if($hash{'crlf'} ||
|
|
+ ($has_hyper && ($keywords{"HTTP"}
|
|
+ || $keywords{"HTTPS"}))) {
|
|
+ map subNewlines(0, \$_), @outfile;
|
|
+ }
|
|
|
|
my $strip;
|
|
for $strip (@stripfile) {
|
|
--
|
|
2.33.0
|
|
|