I would just leave it for now, fixing it has the potential to break applications, with no real gain.StuartOn Thu, 30 Jul 2020 at 13:51, Brad Wood <bdw429s@gmail.com> wrote:Fair enough, is it worth a pull request or do we leave it?Thanks!~BradDeveloper AdvocateOrtus Solutions, CorpE-mail: brad@coldbox.orgColdBox Platform: http://www.coldbox.orgOn Wed, Jul 29, 2020 at 10:47 PM Stuart Douglas <sdouglas@redhat.com> wrote:Probably an oversight, I would guess that maybe there is a TCK test for one method and not the other :-)On Thu, 30 Jul 2020 at 04:07, Brad Wood <bdw429s@gmail.com> wrote:Why does the getResource() method in ServletContextImpl have the following check_______________________________________________if (!path.startsWith("/")) {
throw UndertowServletMessages.MESSAGES.pathMustStartWithSlash(path);
}but the getResourceAsStream() method in the same class does not.The ServletContext spec does say "The path must begin with a / " for the getResource() methodAnd the getResourceAsStream() method also says that "The path must be specified according to the rules given in getResource"So it seems the same validation rules should apply to both methods.Thanks!~BradDeveloper AdvocateOrtus Solutions, CorpE-mail: brad@coldbox.orgColdBox Platform: http://www.coldbox.org
undertow-dev mailing list
undertow-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/undertow-dev