https://issues.apache.org/bugzilla/show_bug.cgi?id=57190

Konstantin Kolinko <knst.koli...@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---

--- Comment #6 from Konstantin Kolinko <knst.koli...@gmail.com> ---
REOPENING. Reviewing r1642697

Using Mapper.map() for this purpose returns wrong result for missing contexts.
The mapper maps prefix of an URL, and so for getContext("/foo/bar") it may
return the context of "/foo" or the ROOT context.

I added a test case in r1643536, but commented-out the check, as it currently
fails.

This needs either a special method in Mapper or in Host (such as
getLatestVersion(contextName)), or a check that the path of returned context is
the same as expected.

A difference between Mapper and Host is that Mapper returns only those contexts
that are available. Using Host may return a context that has been stopped or
have not started yet.  The same concern for the exact match branch of this
method: does it need to return a non-started context, even if the name matches
exactly?

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org

Reply via email to