PATCH: kio_http

[Dawit A.]

On Tuesday 09 April 2002 02:07, Waldo Bastian wrote:
> On Monday 08 April 2002 10:42 pm, Dawit A. wrote:
> > > Yes, please, I did add it for those other methods for a reason.
> >
> > which is ?
>
> They are not idempotent, e.g. if you delete a file twice you will get an
> error the second time, despite that there isn't really an error. All these
> things aren't used very often anyway, so I rather go safe. Matze suggested
> to do the same for GET's with a query but I am not yet convinced of that.
> Maybe only if they are the result of a FORM submit, is there something we
> can check to see if the request was the result of a submit? I guess we
> would need to add extra metadata for that.

That would probably do, but I am not convienced of this either.  Getting an 
error back for the other requests should not be a big problem considering the 
fact that I have yet to see this scenario happen in real world implementation 
unless someone is purposefully trying to deceive people.  But ofcourse it is 
always better to be safe than sorry so I will change that :)

> > Basically no HTTP/1.1 complaint server will send you Connection:
> > Keep-Alive since that is th default.  And it does not work with HTTP/1.0
> > proxies like Squid because we no longer send "Connection: Keep-Alive
> > header".  This issue can really get ugly :)
>
> Maybe we should add an option "I use a broken proxy-server (Junkbuster)"
> then, that way we can use persistent connections with the proxy servers
> that do work.

Indeed, but this needs more thought since doing that right now has not benefit 
especially for those using a 1.0 proxies which many of them out in the wild 
are.  Okay I will make this changes and commit.

Regards,
Dawit A.