Review Request: kio_http: fix keepalive timeout parsing
Rolf Eike Beer
kde at opensource.sf-tec.de
Tue Oct 11 11:27:26 BST 2011
> Testing
> -------
>
> -Patched code compiles
> -Hacked a web server and made tests against following keep-alive header
> variants:
> "Keep-Alive: timeout=5, max=99"
> "Keep-Alive: Timeout=5, max=99" (uppercase 'T')
> "Keep-Alive: Timeout=5 , max=99" (extra space before comma)
I don't know which RfC this comes from and if that makes any requirements
about the ordering, but other header fields (like Content-Disposition) do
not have some. So what happens with e.g.:
max=99, timeout=5
max = 99, timeout = 5
foo = bar, timeout = 5
Is it possible to add a unit test for this?
Eike
More information about the kde-core-devel
mailing list