On Fri, Jan 27, 2023 at 2:51 PM <ma...@apache.org> wrote: > > This is an automated email from the ASF dual-hosted git repository. > > markt pushed a commit to branch main > in repository https://gitbox.apache.org/repos/asf/tomcat.git > > commit ce8621750e4f74229bffa7e42e648523b91b91de > Author: Mark Thomas <ma...@apache.org> > AuthorDate: Fri Jan 27 12:57:39 2023 +0000 > > Redirect no longer required for root mapping
But this would break directory listing for the webapp root right ? Rémy > --- > .../servlet/annotation/TestServletSecurityMappings.java | 10 > +++------- > 1 file changed, 3 insertions(+), 7 deletions(-) > > diff --git a/test/jakarta/servlet/annotation/TestServletSecurityMappings.java > b/test/jakarta/servlet/annotation/TestServletSecurityMappings.java > index 60440ea0fe..e438ae563e 100644 > --- a/test/jakarta/servlet/annotation/TestServletSecurityMappings.java > +++ b/test/jakarta/servlet/annotation/TestServletSecurityMappings.java > @@ -118,14 +118,10 @@ public class TestServletSecurityMappings extends > TomcatBaseTest { > > // Root > rc = getUrl("http://localhost:" + getPort() + "/test", bc, false); > - if (redirectContextRoot) { > - Assert.assertEquals(302, rc); > + if (secureRoot || secureDefault) { > + Assert.assertEquals(403, rc); > } else { > - if (secureRoot || secureDefault) { > - Assert.assertEquals(403, rc); > - } else { > - Assert.assertEquals(200, rc); > - } > + Assert.assertEquals(200, rc); > } > } > > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org > For additional commands, e-mail: dev-h...@tomcat.apache.org > --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org