Daniel Stenberg [Wed, 30 Dec 2009 21:52:27 +0000 (21:52 +0000)]
(SMTP) support DATA better in the server and make sure to "escape" CRLF.CRLF
sequences in uploaded data. The test server doesn't "decode" escaped dot-lines
but instead test cases must be written to take them into account. Added test
case 803 to verify dot-escaping.
Yang Tse [Tue, 29 Dec 2009 01:51:57 +0000 (01:51 +0000)]
- Where run-time error checks enabling compiler option /GZ was used it is now
replaced with equivalent /RTCsu for Visual Studio 2003 and newer versions.
- Compiler option /GX is now replaced with equivalent /EHsc for all versions.
Daniel Stenberg [Fri, 25 Dec 2009 22:21:46 +0000 (22:21 +0000)]
When verifying the SMTP server I had to add some hackery since SMTP in itself
doesn't really return any body data to check so I made SMTP check "headers"
as well.
Yang Tse [Wed, 23 Dec 2009 15:20:20 +0000 (15:20 +0000)]
Back out some pingpong server stopping changes introduced last week in
order to re-use 'classic' stopping and allow taking test-suite run time
references from this state.
Yang Tse [Mon, 21 Dec 2009 16:55:39 +0000 (16:55 +0000)]
- Ingmar Runge noticed that Windows config-win32.h configuration file
did not include a definition for HAVE_CLOSESOCKET which resulted in
function close() being inappropriately used to close sockets.
Yang Tse [Mon, 21 Dec 2009 14:53:07 +0000 (14:53 +0000)]
Move stopping of servers instructed by test definition, after the removal of
server logs advisor read lock and before test results verifications are done.
When stopping a pingpong server, stop server before its slaves.
Yang Tse [Mon, 21 Dec 2009 14:33:01 +0000 (14:33 +0000)]
Reinstate hi resolution time logging since this is not what is
now adding 90 extra seconds to the total time some autobuilds
need to make a full test-run.
Yang Tse [Thu, 17 Dec 2009 22:28:40 +0000 (22:28 +0000)]
Comment out hi resolution time logging to verify if this is what might
be contributing 90 additional seconds to the total time some autobuilds
now need to make a full test-run.
Daniel Stenberg [Wed, 16 Dec 2009 23:11:47 +0000 (23:11 +0000)]
Follow-up fix for the proxy fix I did for Jon Nelson's bug. It turned out I
was a bit too quick and broke test case 1101 with that change. The order of
some of the setups is sensitive. I now changed it slightly again.
Daniel Stenberg [Mon, 14 Dec 2009 23:16:09 +0000 (23:16 +0000)]
- Jon Nelson found a regression that turned out to be a flaw in how libcurl
detects and uses proxies based on the environment variables. If the proxy
was given as an explicit option it worked, but due to the setup order
mistake proxies would not be used fine for a few protocols when picked up
from '[protocol]_proxy'. Obviously this broke after 7.19.4. I now also added
test case 1106 that verifies this functionality.
Yang Tse [Mon, 14 Dec 2009 15:39:15 +0000 (15:39 +0000)]
Prevent perl script dying messages in output, since tearing down the pinpong
server in this way, upon sysread failures, is part of the expected behavior.
Daniel Stenberg [Thu, 10 Dec 2009 21:02:11 +0000 (21:02 +0000)]
- Siegfried Gyuricsko found out that the curl manual said --retry would retry
on FTP errors in the transient 5xx range. Transient FTP errors are in the
4xx range. The code itself only tried on 5xx errors that occured _at login_.
Now the retry code retries on all FTP transfer failures that ended with a
4xx response.
Daniel Stenberg [Thu, 10 Dec 2009 20:20:15 +0000 (20:20 +0000)]
- Constantine Sapuntzakis figured out a case which would lead to libcurl
accessing alredy freed memory and thus crash when using HTTPS (with
OpenSSL), multi interface and the CURLOPT_DEBUGFUNCTION and a certain order
of cleaning things up. I fixed it.
Daniel Stenberg [Mon, 7 Dec 2009 20:25:17 +0000 (20:25 +0000)]
- Martin Storsjo made libcurl use the Expect: 100-continue header for posts
with unknown size. Previously it was only used for posts with a known size
larger than 1024 bytes.
Daniel Stenberg [Fri, 4 Dec 2009 21:52:58 +0000 (21:52 +0000)]
74. The HTTP spec allows headers to be merged and become comma-separated
instead of being repeated several times. This also include Authenticate: and
Proxy-Authenticate: headers and while this hardly every happens in real life
it will confuse libcurl which does not properly support it for all headers -
like those Authenticate headers.
Daniel Stenberg [Tue, 1 Dec 2009 12:04:54 +0000 (12:04 +0000)]
- If the Expect: 100-continue header has been set by the application through
curl_easy_setopt with CURLOPT_HTTPHEADER, the library should set
data->state.expect100header accordingly - the current code (in 7.19.7 at
least) doesn't handle this properly. Martin Storsjo provided the fix!
Yang Tse [Mon, 30 Nov 2009 13:48:51 +0000 (13:48 +0000)]
- In order to better reflect that the returned pid is extracted from the
given file, serverpid sub is renamed to pidfromfile. In addition it is
enhanced to make sure that it always returns zero unless a numerical
positive value is returned.
- To better reflect that only process existance is actually checked,
checkserver sub is renamed to processexists. In addition it is enhanced
making it remove the given pid file when the extracted pid is no longer
alive.