That'll have to be be changed in the client library. I can bring it up and provide a patch, but Funambol is currently migrating their source code to SVN. It'll be a while before I can release an update (perhaps a month from now).
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
I installed the latest SyncEvolution deb package from http://www.estamos.de/download/apt (Package syncevolution-evolution, version 1:0.8.1-2) but I think I am still seeing this error. I'm running Ubuntu Intrepid.
$ syncevolution --version
SyncEvolution 0.8.1
using libedataserver-1.2.so.11
using libebook-1.2.so.9
using libecal-1.2.so.7
Do I need to install from source to get the fix for this? I took a look at the source for syncevolution-0.8.1 and the file src/client-api/src/c++/posix/http/CurlTransportAgent.cpp still has a reference to the Expect header in line 81:
slist = curl_slist_append(NULL, "Expect:");
Any ideas?
Richard
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
> I took a look
> at the source for syncevolution-0.8.1 and the file
> src/client-api/src/c++/posix/http/CurlTransportAgent.cpp still has a
> reference to the Expect header in line 81:
> slist = curl_slist_append(NULL, "Expect:");
That is the fix. Listing the value explicitly with no value suppresses generating the "Expect: 100". It worked with libcurl 7.15.5-1etch1 (Debian Etch), but now that you have mentioned it, I also checked with libcurl7.18.0-1ubuntu2 (Ubuntu 8.04) and there I can reproduce that the problem is back.
I need to check whether that is a regression in libcurl and how this can be worked around.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
I'm no longer sure whether the "fix" ever worked. The explicit "Expect:" header was overwritten later on. I fixed the issue properly this time around. Not sure why I missed it when testing the first patch - sorry! :-/
Richard, I'm not sure when I'll be able to put out new binaries. In the meantime please compile from source after applying the attached patch: in the src/client-api directory use "patch -p1 < 'name of patch file'"
File Added: curl-expect-II.patch
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
Awesome, that's fixed it for me. I thought it made more sense to set the header values in sendMessage() than in the constructor. Thanks for the quick fix.
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
That'll have to be be changed in the client library. I can bring it up and provide a patch, but Funambol is currently migrating their source code to SVN. It'll be a while before I can release an update (perhaps a month from now).
Okey... let me know if I can help.
Information:
Ubuntu 8.04.1
Evolution 2.22
SyncEvolution 0.8
Our company proxy (squid 2.7) also produce 417 error on "Expect: 100-continue"
I can't sync my contacts :(
I have a patch for this and will it use in the upcoming 0.8.1 (if all goes well, due this week or the next).
The patch currently is here:
http://github.com/pohly/funambol-cpp-client-api/commit/a42b7f79dc720ad2becdb760f413c033366c2a00
Patch also included upstream.
Hi,
I installed the latest SyncEvolution deb package from http://www.estamos.de/download/apt (Package syncevolution-evolution, version 1:0.8.1-2) but I think I am still seeing this error. I'm running Ubuntu Intrepid.
$ syncevolution --version
SyncEvolution 0.8.1
using libedataserver-1.2.so.11
using libebook-1.2.so.9
using libecal-1.2.so.7
My client.log shows the following:
...
[DEBUG] header out: User-Agent: SyncEvolution
[DEBUG] header out: Host: [snip]
[DEBUG] header out: Accept: */*
[DEBUG] header out: Content-Type: application/vnd.syncml+xml
[DEBUG] header out: Content-Length: 2019
[DEBUG] header out: Expect: 100-continue
[DEBUG] header out:
20:57:05 GMT [DEBUG] header in: HTTP/1.1 417 Expectation Failed
20:57:05 GMT [DEBUG] header in: Connection: close
20:57:05 GMT [DEBUG] header in: Content-Length: 0
20:57:05 GMT [DEBUG] header in: Date: Thu, 01 Jan 2009 20:57:05 GMT
20:57:05 GMT [DEBUG] header in: Server: lighttpd/1.4.18
20:57:05 GMT [DEBUG] header in:
20:57:05 GMT [DEBUG] libcurl info: Closing connection #0
...
Do I need to install from source to get the fix for this? I took a look at the source for syncevolution-0.8.1 and the file src/client-api/src/c++/posix/http/CurlTransportAgent.cpp still has a reference to the Expect header in line 81:
slist = curl_slist_append(NULL, "Expect:");
Any ideas?
Richard
> I took a look
> at the source for syncevolution-0.8.1 and the file
> src/client-api/src/c++/posix/http/CurlTransportAgent.cpp still has a
> reference to the Expect header in line 81:
> slist = curl_slist_append(NULL, "Expect:");
That is the fix. Listing the value explicitly with no value suppresses generating the "Expect: 100". It worked with libcurl 7.15.5-1etch1 (Debian Etch), but now that you have mentioned it, I also checked with libcurl7.18.0-1ubuntu2 (Ubuntu 8.04) and there I can reproduce that the problem is back.
I need to check whether that is a regression in libcurl and how this can be worked around.
I'm no longer sure whether the "fix" ever worked. The explicit "Expect:" header was overwritten later on. I fixed the issue properly this time around. Not sure why I missed it when testing the first patch - sorry! :-/
Richard, I'm not sure when I'll be able to put out new binaries. In the meantime please compile from source after applying the attached patch: in the src/client-api directory use "patch -p1 < 'name of patch file'"
File Added: curl-expect-II.patch
Awesome, that's fixed it for me. I thought it made more sense to set the header values in sendMessage() than in the constructor. Thanks for the quick fix.