tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Brian Burch <>
Subject Re: [Bug 54190] TestNonLoginAndBasicAuthenticator does not test session timeout properly
Date Thu, 29 Nov 2012 16:11:58 GMT
On 29/11/12 14:37, wrote:
> Mark Thomas <> changed:
>             What    |Removed                     |Added
> ----------------------------------------------------------------------------
>               Status|NEW                         |RESOLVED
>           Resolution|---                         |FIXED
> --- Comment #6 from Mark Thomas <> ---
> Thanks. Patch applied to trunk and 7.0.x and will be included in 7.0.34
> onwards.
> Applying the patch generated a bunch of IDE warnings (we try to keep the code
> clean of those) which were fixed by r1415178 and r1415179.

Thanks Mark.

I use netbeans, not eclipse. My change was clear of ide warnings and it 
also passed checkstyle.

When I looked quickly at your change, the "-" and "+" lines appeared to 
be identical to me, so I was puzzled.

Could you give me an example of what kind of warnings you were getting - 
I noticed that you commented on having more than 40, so just the most 
common one or two would be a help.

Curiously, you haven't mentioned this kind of problem with my previous 
patches, and all have been generated as svn diffs rather than ide diffs. 
However, I have recently upgraded both netbeans and svn on my main system.

I probably need to adjust one of my netbeans settings to conform.

> If you start looking at the TODOs, I suggest you take a look at
> org.apache.tomcat.util.http.parser.HttpParser#parseAuthorizationDigest()
> I suspect a new parseAuthorizationBasic() method is the way to go as that
> should handle the various whitespace issues noted.

Noted. Assume that I will look into it unless you hear otherwise. I 
won't open a bug against BasicAuthenticator yet.



To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message