Re: [hail patch 1/1] Make host, url, orig_path dynamic

Jeff Garzik <[email protected]> Tue, 20 Jul 2010 16:34:19 -0400
Newsgroups org.kernel.vger.hail-devel
Message-ID <[email protected]>
On 07/20/2010 04:16 PM, Pete Zaitcev wrote:
> Some of my performance tests for tabled hit truncation again:
>
>   [zaitcev@hitlain tests]$ ./poke5 -v -h niphredil.zaitcev.lan -u auser
>    -p apass -b test -o -k testkey-hitlain/73b84a11e6d83c65e45853338d646042
>    -f testdir/73b84a11e6d83c65e45853338d646042
>   * About to connect() to niphredil.zaitcev.lan port 80 (#0)
>   *   Trying fec0::1:219:b9ff:fe58:7ad6... * TCP_NODELAY set
>   * connected
>   * Connected to niphredil.zaitcev.lan (fec0::1:219:b9ff:fe59:7ad6) port 80 (#0)
>   >  PUT /test/testkey-hitlain/73b84a11e6d83c65e45853338d HTTP/1.1
>   Accept: */*
>   Host: niphredil.zaitcev.lan
>   Date: Tue, 20 Jul 2010 01:07:33 +0000
>   Authorization: AWS testuser:RefcbVYgr2m9KTRxOrCfr4zzfPE=
>   Content-Length: 214745088
>   Expect: 100-continue
>
>   * The requested URL returned error: 403
>
> As you can see, the path in PUT is truncated, and this causes 403
> since it's included into a hash.
>
> The patch addresses this issue and a bunch of other fixed-size
> strings before we hit that.
>
> Signed-off-by: Pete Zaitcev<[email protected]>
>
> ---
>   lib/hstor.c |  147 +++++++++++++++++++++++++++++++++++---------------
>   1 file changed, 104 insertions(+), 43 deletions(-)

applied