Thierry,

I have come up with a "fix" that solves the issue for us... but I don't know if 
it is the best/right solution.  It doesn't seem to be the right way to handle 
it.


I'm using the 2.0.11 source.

In HttpMethodCall.sendRequest(Request) I noticed that after:

    this.httpResponse = this.clientHelper.getHttpClient().execute(
                    getHttpRequest());

when catching the exception they do this:

            // Release the connection
            getHttpRequest().abort();

So in the non-exception case I added a test:

            if (result.isError()) {
                getHttpRequest().abort();
            }

It seems odd to me that this code should have to know to abort the request in 
these error & exception cases, but not in the successful case.  I also wonder 
if there are other exceptions (e.g. ClientProtocolException) that should be 
caught and handled similarly.

------------------------------------------------------
http://restlet.tigris.org/ds/viewMessage.do?dsForumId=4447&dsMessageId=2994842

Reply via email to