Re: [curl:bugs] #1183 curl FTP upload performance degration version 7.28.1

2013-01-17 Thread Daniel Stenberg
On Thu, 17 Jan 2013, Daniel Theron wrote: I am in the process upgrading curl from version 7.19.6 to version 7.28.1 on our Solaris 10 servers. I have written a unix script to do file upload tests with ftp sftp, and with different file types like binary and ascii (dos and unix). With the

[PATCH] Addition of trailer headers in HTTP requests generated by libcurl

2013-01-17 Thread Chrysovaladis Datsios
Hi, this is a patch for the feature request as described in http://sourceforge.net/p/curl/feature-requests/67/ the feature is a standard in http://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.40 One can also refer at Chris Shiflett's Book HTTP: Developer's Handbook from SAMS page 97.

Re: [curl:bugs] #1183 curl FTP upload performance degration version 7.28.1

2013-01-17 Thread Tor Arntsen
On Thu, Jan 17, 2013 at 1:32 PM, Daniel Stenberg dan...@haxx.se wrote: On Thu, 17 Jan 2013, Daniel Theron wrote: I am in the process upgrading curl from version 7.19.6 to version 7.28.1 on our Solaris 10 servers. I have written a unix script to do file upload tests with ftp sftp, and with

Can I post my open source library?

2013-01-17 Thread John Kenedy
Before I post I want to make sure admin is okay with it. So its like this, I have extend microsoft .NET's WebClient class and build a syntax parser that is specially for web scraping functions. Can I post my open source library link here, so everyone can take a look and probably inspire me to

Re: [PATCH] always multi v5

2013-01-17 Thread Yang Tse
Commit https://github.com/bagder/curl/commit/9fd88abb pushed now for autobuild testing, solves the problems shown locally with test cases 1059 and 1316. The patch has been tested without 'always multi' on top of commit 4ed6b07a, with test suite passing ok. Additionally, patch tested with 'always

Change .gitignore for building w/ Windows and VC++

2013-01-17 Thread Anthony Perez
I'm using curl as a submodule in a project of mine, and following the build instructions in winbuild/BUILD.WINDOWS.txt, I get a couple new *.h files under the include/curl directory for the corresponding build directory. At that point git thinks there's new files it needs to track for curl, and

Re: [PATCH] always multi v5

2013-01-17 Thread Daniel Stenberg
On Thu, 17 Jan 2013, Yang Tse wrote: Commit https://github.com/bagder/curl/commit/9fd88abb pushed now for autobuild testing, solves the problems shown locally with test cases 1059 and 1316. Currently I have no functional pending issues with the 'always multi' modification. Great. I intend

Re: Can I post my open source library?

2013-01-17 Thread Dan Fandrich
On Thu, Jan 17, 2013 at 09:05:54PM +0800, John Kenedy wrote: Before I post I want to make sure admin is okay with it. So its like this, I have extend microsoft .NET's WebClient class and build a syntax parser that is specially for web scraping functions. Can I post my open source library

Re: [PATCH] Addition of trailer headers in HTTP requests generated by libcurl

2013-01-17 Thread Dan Fandrich
On Thu, Jan 17, 2013 at 02:35:12PM +0200, Chrysovaladis Datsios wrote: The feature is implemented as another in curl_easy_setopt() function. This new option is named CURLOPT_HTTPTRAILERHEADER and adds the trailer headers appended on a curl_slist. Example code snippet using the new feature:

curl_easy_perform stuck in select

2013-01-17 Thread Brett Schoppert
What could cause my (mac) application to become stuck in select during a call to curl_easy_perform ? Which timeout would I need to set to avoid this ? + 8647 Curl_do_perform (in libhorizon-api.dylib) + 795 [0x100e646cb] + 8647 Curl_socket_check (in libhorizon-api.dylib) + 527 [0x100e705ff]

Re: [PATCH] Addition of trailer headers in HTTP requests generated by libcurl

2013-01-17 Thread Daniel Stenberg
On Thu, 17 Jan 2013, Dan Fandrich wrote: +char tfield[CURL_MAX_HTTP_HEADER]; +strncpy(tfield, trailer_headers-data, tptr-trailer_headers-data+1); This will overflow tfield given a long enough user-supplied header. In addition to Dan's fine comments, allow me to point out

Re: [PATCH] always multi v5

2013-01-17 Thread Daniel Stenberg
On Thu, 17 Jan 2013, Daniel Stenberg wrote: Great. I intend to merge my always-multi patch now. Or at least during this evening. Thanks for all your help with this! And it has been pushed as commit c43127414d89ccb9. A repeated lengthier explanation of the work:

Re: cURL behavior on 303 does not seem to be what is documented/advertised?

2013-01-17 Thread Daniel Stenberg
On Wed, 16 Jan 2013, Yves Arrouye wrote: You hardly ever truly want to use -X. Ok. There are thousands of examples showing that (see resty etc.). But if you look again, you'll see that basically none of them are from anyone in the curl project or are hosted on the curl site. A bad idea

Re: cURL behavior on 303 does not seem to be what is documented/advertised?

2013-01-17 Thread Yves Arrouye
Thanks. The man page on -X is unclear: it does not mention that -X is sticky across redirections. Maybe that could be improved. The verbose message saying using GET right before POSTing could also be slightly conditional so it says the right thing :) You should instead get the host name from the

Re: cURL behavior on 303 does not seem to be what is documented/advertised?

2013-01-17 Thread Yves Arrouye
What about PUT? That's what -T makes. This requires file names, correct? Maybe that's why people using curl for REST services start with a -X PUT and then just go -X method for everything? YA ­­­ Learn about GPT services and architectures on Confluence.

CMake: OpenSSL updates

2013-01-17 Thread Sergei Nikulov
Hello All! I'm using cmake to build curl/libcurl on Windows/Linux. The problem I've found on Windows - cmake incorrectly produce curl_config.h even if openssl found. For Linux it is not so visible because openssl includes resides in default location and cmake will find them anyway. But issue can

Re: cURL behavior on 303 does not seem to be what is documented/advertised?

2013-01-17 Thread Daniel Stenberg
On Thu, 17 Jan 2013, Yves Arrouye wrote: Thanks. The man page on -X is unclear: it does not mention that -X is sticky across redirections. Maybe that could be improved. Everything can be improved. Did you really read this documentation first and then yet beleived -X POST was what you needed?