You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@tomcat.apache.org by bu...@apache.org on 2017/01/03 11:57:52 UTC

[Bug 60488] NPE in Request.parseCookies()

https://bz.apache.org/bugzilla/show_bug.cgi?id=60488

Mark Thomas <ma...@apache.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |INVALID
             Status|NEW                         |RESOLVED

--- Comment #6 from Mark Thomas <ma...@apache.org> ---
Yes, it is expected that the Connector is non-null.

Looking at the code, we could make the connector field final and requiring it
in the constructor. We might be able to go further and require a non-null
Connector is provided. That would require refactoring a few tests but the
changes to the Tomcat code would be minimal. I'll look at this for trunk
(9.0.x). Whether I actually commit it will depend on what the refactoring ends
up looking like and if there are any side-effects I haven't spotted yet.

-- 
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org