Re: [jetty-discuss] [PATCH] Fix trailing whitespace in form requests

[Greg W. <gr...@mo...>]

je...@i3... wrote:

>>The buf that is being exposes is part of the standard
>>API for the class, so it should be portable.
>  
> True.  Except that the javadocs say nothing about the length of "buf".

But it does expose count, which I'm using.


>>I'll work on a patch for this shortly.

The fix to use the count is checked into head hand Jetty_4_2 branch now.
Releases out this weekend.


>>Note that it the grand scheme of things, I do agree with you that
>>the approach is wrong.  Ideally I would like to parse the form
>>content as it arrives and not have to have it completely in a buffer
>>at all (thus I could have a fixed size buffer that is reused).
>>I'll be looking at doing this in JettyExperimental.
> 
> Sounds good.  I ought to take a look at JettyExperimental sometime, as it
> sounds like a good fit for what we do.

It's getting close to usable now...   If I can find another day or two
spare I hope to do an alpha release of it.

cheers