Hi Mark, On Wed, Feb 19, 2020 at 3:09 PM Mark Thomas <ma...@apache.org> wrote:
> On 19/02/2020 12:37, mgrigo...@apache.org wrote: > > This is an automated email from the ASF dual-hosted git repository. > > > > mgrigorov pushed a change to branch master > > in repository https://gitbox.apache.org/repos/asf/tomcat.git. > > > > > > from 81cfd2d Disable session persistence by default on > StandardManager > > add 9ff6b12 Check for non-null 'methods' > > Please revert. This check is unnecessary. methods cannot be null here. > https://github.com/apache/tomcat/blob/292b744994cf0cddf86b651dc1b1935dbe4fc83f/java/jakarta/servlet/http/HttpServlet.java#L514-L516 Here getAllDeclaredMethods() returns `null`. Maybe it should return an empty array ? > > Mark > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org > For additional commands, e-mail: dev-h...@tomcat.apache.org > >
