users@servlet-spec.java.net

[servlet-spec users] Re: Implementation of new Servlet functionality?

From: Stuart Douglas <stuart.w.douglas_at_gmail.com>
Date: Wed, 06 Apr 2016 23:14:19 +0000

At the moment I am not treating JSP specially, so we just return whatever
the mapping type is for the original Servlet mapping.

I don't really see any advantage in treating JSP's as a special case.

Stuart

On Thu, 7 Apr 2016 at 09:02 Greg Wilkins <gregw_at_webtide.com> wrote:

>
> What is the mapping for file mapped JSPs?
>
>
> On 7 April 2016 at 03:48, arjan tijms <arjan.tijms_at_gmail.com> wrote:
>
>> Hi,
>>
>> On Wednesday, April 6, 2016, Paul Benedict <pbenedict_at_apache.org> wrote:
>>
>>> Question on the mapping. If a request comes into a servlet and the
>>> servlet includes/forwards to a JSP, what does the JSP see? I presume it
>>> will be the Mapping of the implicit *.jsp servlet, correct?
>>>
>>
>> Indeed, that's happening now.
>>
>>
>>
>>> Will there be any special request attributes to expose the servlet's --
>>> something to the effect like the javax.servlet.include/forward attributes?
>>>
>>
>> That's the plan it seems, just discussed a couple of messages back ;)
>>
>> Kind regards,
>> Arjan Tijms
>>
>>
>>
>>>
>>> Cheers,
>>> Paul
>>>
>>> On Wed, Apr 6, 2016 at 11:36 AM, arjan tijms <arjan.tijms_at_gmail.com>
>>> wrote:
>>>
>>>> On Wed, Apr 6, 2016 at 6:32 PM, Mark Thomas <markt_at_apache.org> wrote:
>>>>
>>>>> Lets remove that for now then. It is much easier to add it later if a
>>>>> use is found for it than it is to remove something that turns out to be
>>>>> pointless.
>>>>>
>>>>
>>>> +1
>>>>
>>>>
>>>>
>>>>>
>>>>> > >>
>>>>> > >> Mark
>>>>> > >>
>>>>> > >>
>>>>> > >> >
>>>>> > >> > Stuart
>>>>> > >> >
>>>>> > >> > On Mon, Apr 4, 2016 at 10:19 PM, arjan tijms
>>>>> > ><arjan.tijms_at_gmail.com <mailto:arjan.tijms_at_gmail.com>>
>>>>> > >> wrote:
>>>>> > >> >> Hi,
>>>>> > >> >>
>>>>> > >> >> I wrote a small blog post about the new API here:
>>>>> > >> >>
>>>>> > >>
>>>>> > >
>>>>> http://arjan-tijms.omnifaces.org/2016/04/servlet-40s-mapping-api-previewed-in.html
>>>>> > >> >>
>>>>> > >> >> I think we need to discuss the "implicit" mapping. As by my
>>>>> > >experiments,
>>>>> > >> >> Tomcat never seems to return it, even when calling a *.jsp
>>>>> page.
>>>>> > >It's
>>>>> > >> >> debatable if implicit even is a separate mapping. I mean,
>>>>> can't we
>>>>> > >have
>>>>> > >> an
>>>>> > >> >> implicit path mapping, implicit extension mapping etc?
>>>>> Maybe it's
>>>>> > >> better to
>>>>> > >> >> have an isImplicit() method on the Mapping type?
>>>>> > >> >>
>>>>> > >> >> I also got some feedback regarding the getPattern()
>>>>> method. People
>>>>> > >> (also)
>>>>> > >> >> seem to want the pattern in such a way that they can use
>>>>> it right
>>>>> > >away
>>>>> > >> in
>>>>> > >> >> their code. E.g. for path mapping the pattern would now be
>>>>> e.g.
>>>>> > >> "/path/*",
>>>>> > >> >> but it would also be desirable to have a "/path/" returned.
>>>>> > >> >>
>>>>> > >> >> Another comment concerned the Default mapping. It's now
>>>>> e.g.:
>>>>> > >> >>
>>>>> > >> >> Default (fallback) mapping
>>>>> > >> >> http://localhost:8080/servlet4/doesnotexist
>>>>> > >> >>
>>>>> > >> >> Mapping match:DEFAULT
>>>>> > >> >> Match value:/
>>>>> > >> >> Pattern:/
>>>>> > >> >>
>>>>> > >> >> Should the match value be "/" here, or perhaps better be
>>>>> > >> "/doesnotexist"?
>>>>> > >> >>
>>>>> > >> >> What do you think?
>>>>> > >> >>
>>>>> > >> >> Kind regards,
>>>>> > >> >> Arjan Tijms
>>>>> > >> >>
>>>>> > >> >>
>>>>> > >> >>
>>>>> > >> >>
>>>>> > >> >>
>>>>> > >> >>
>>>>> > >> >>
>>>>> > >> >> On Thu, Mar 31, 2016 at 9:42 AM, arjan tijms
>>>>> > ><arjan.tijms_at_gmail.com <mailto:arjan.tijms_at_gmail.com>>
>>>>> > >> wrote:
>>>>> > >> >>>
>>>>> > >> >>> Great, thanks!
>>>>> > >> >>>
>>>>> > >> >>> On Thu, Mar 31, 2016 at 12:58 AM, Greg Wilkins
>>>>> > ><gregw_at_webtide.com <mailto:gregw_at_webtide.com>>
>>>>> > >> wrote:
>>>>> > >> >>>>
>>>>> > >> >>>> It is on the todo list.... but probably at least 2 weeks
>>>>> away.
>>>>> > >> >>>>
>>>>> > >> >>>> cheers
>>>>> > >> >>>>
>>>>> > >> >>>>
>>>>> > >> >>>> On 31 March 2016 at 09:54, arjan tijms
>>>>> > <arjan.tijms_at_gmail.com <mailto:arjan.tijms_at_gmail.com>>
>>>>> > >wrote:
>>>>> > >> >>>>>
>>>>> > >> >>>>> Greg, Stuart,
>>>>> > >> >>>>>
>>>>> > >> >>>>> Would it be possible for you to also implement the
>>>>> getMapping
>>>>> > >API
>>>>> > >> that
>>>>> > >> >>>>> Mark implemented for Tomcat? That may help ironing out
>>>>> some
>>>>> > >> potential issues
>>>>> > >> >>>>> as Mark mentioned
>>>>> > >> >>>>>
>>>>> > >> >>>>> Kind regards,
>>>>> > >> >>>>> Arjan Tijms
>>>>> > >> >>>>>
>>>>> > >> >>>>>
>>>>> > >> >>>>>
>>>>> > >> >>>>> On Thu, Mar 31, 2016 at 12:18 AM, Greg Wilkins
>>>>> > ><gregw_at_webtide.com <mailto:gregw_at_webtide.com>>
>>>>> > >> >>>>> wrote:
>>>>> > >> >>>>>>
>>>>> > >> >>>>>>
>>>>> > >> >>>>>> All,
>>>>> > >> >>>>>>
>>>>> > >> >>>>>> Jetty's stable releases (9.3.x) have had PushBuilder
>>>>> support
>>>>> > >for a
>>>>> > >> few
>>>>> > >> >>>>>> months and it is actively being used by our HTTP2
>>>>> adopters. It
>>>>> > >was
>>>>> > >> this
>>>>> > >> >>>>>> usage that found the race condition issues and
>>>>> transient push
>>>>> > >issues
>>>>> > >> >>>>>> discussed elsewhere.
>>>>> > >> >>>>>>
>>>>> > >> >>>>>> Other that that, no complaints so far. However, this
>>>>> is not
>>>>> > >> against a
>>>>> > >> >>>>>> servlet 4.0 API but rather our own packaging of the
>>>>> same API.
>>>>> > >> >>>>>>
>>>>> > >> >>>>>> cheers
>>>>> > >> >>>>>>
>>>>> > >> >>>>>>
>>>>> > >> >>>>>> On 31 March 2016 at 08:38, Stuart Douglas
>>>>> > ><sdouglas_at_redhat.com <mailto:sdouglas_at_redhat.com>>
>>>>> > >> wrote:
>>>>> > >> >>>>>>>
>>>>> > >> >>>>>>> I have initial PushBuilder support implemented in
>>>>> Undertow
>>>>> > >master,
>>>>> > >> >>>>>>> however it is not part of any release yet. It should
>>>>> be
>>>>> > >possible to
>>>>> > >> >>>>>>> use it in Wildfly by simply replacing the existing
>>>>> Undertow
>>>>> > >and
>>>>> > >> >>>>>>> Servlet API jars.
>>>>> > >> >>>>>>>
>>>>> > >> >>>>>>> Stuart
>>>>> > >> >>>>>>>
>>>>> > >> >>>>>>> On Wed, Mar 30, 2016 at 10:02 PM, arjan tijms <
>>>>> > >> arjan.tijms_at_gmail.com <mailto:arjan.tijms_at_gmail.com>>
>>>>> > >> >>>>>>> wrote:
>>>>> > >> >>>>>>>> Hi,
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>> On Wed, Mar 30, 2016 at 10:04 AM, Mark Thomas
>>>>> > ><markt_at_apache.org <mailto:markt_at_apache.org>>
>>>>> > >> >>>>>>>> wrote:
>>>>> > >> >>>>>>>>>
>>>>> > >> >>>>>>>>> Do you mean this one:
>>>>> > >> >>>>>>>>>
>>>>> > >> >>>>>>>>>
>>>>> > >> >>>>>>>>>
>>>>> > >>
>>>>> > >
>>>>> http://search.maven.org/#artifactdetails|org.apache.tomcat|tomcat-servlet-api|9.0.0.M4|jar
>>>>> > >> >>>>>>>>>
>>>>> > >> >>>>>>>>> All the Tomcat 9.0.0.M4 JARs should be in Maven
>>>>> central
>>>>> > >along
>>>>> > >> with
>>>>> > >> >>>>>>>>> a
>>>>> > >> >>>>>>>>> full installer as well.
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>> Perfect, thanks!
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>>>
>>>>> > >> >>>>>>>>>> A question is how to proceed with this feature for
>>>>> other
>>>>> > >Servlet
>>>>> > >> >>>>>>>>>> containers, but that's probably best discussed at
>>>>> the
>>>>> > >Servlet EG
>>>>> > >> >>>>>>>>>> list.
>>>>> > >> >>>>>>>>>
>>>>> > >> >>>>>>>>> The EG members should be watching here...
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>> You're right, they should be.
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>>>
>>>>> > >> >>>>>>>>>
>>>>> > >> >>>>>>>>> I think the first step is for other containers to
>>>>> implement
>>>>> > >the
>>>>> > >> >>>>>>>>> proposal
>>>>> > >> >>>>>>>>> so we can iron out any ambiguities in the spec and
>>>>> any
>>>>> > >> >>>>>>>>> implementation
>>>>> > >> >>>>>>>>> issues we didn't hit in Tomcat.
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>> Sounds like a very good plan indeed. I guess Jetty
>>>>> and
>>>>> > >Undertow as
>>>>> > >> >>>>>>>> active
>>>>> > >> >>>>>>>> open source containers would be good initial
>>>>> candidates
>>>>> > >here?
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>> Kind regards,
>>>>> > >> >>>>>>>> Arjan Tijms
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>>>>
>>>>> > >> >>>>>>>>>
>>>>> > >> >>>>>>>>> Mark
>>>>> > >> >>>>>>>>>
>>>>> > >> >>>>>>>>>
>>>>> > >> >>>>>>>>>> Kind regards,
>>>>> > >> >>>>>>>>>> Arjan Tijms
>>>>> > >> >>>>>>>>>>
>>>>> > >> >>>>>>>>>>
>>>>> > >> >>>>>>>>>>
>>>>> > >> >>>>>>>>>> On Tue, Mar 29, 2016 at 8:26 AM, Mark Thomas
>>>>> > ><markt_at_apache.org <mailto:markt_at_apache.org>
>>>>> > >> >>>>>>>>>> <mailto:markt_at_apache.org <mailto:markt_at_apache.org
>>>>> >>>
>>>>> > wrote:
>>>>> > >> >>>>>>>>>>
>>>>> > >> >>>>>>>>>> Any feedback on this?
>>>>> > >> >>>>>>>>>>
>>>>> > >> >>>>>>>>>> Mark
>>>>> > >> >>>>>>>>>>
>>>>> > >> >>>>>>>>>>
>>>>> > >> >>>>>>>>>> On 17/03/2016 10:39, arjan tijms wrote:
>>>>> > >> >>>>>>>>>> > Hi,
>>>>> > >> >>>>>>>>>> >
>>>>> > >> >>>>>>>>>> > That's really great! I'll do some testing
>>>>> over the
>>>>> > >weekend
>>>>> > >> >>>>>>>>>> and let
>>>>> > >> >>>>>>>>>> you know.
>>>>> > >> >>>>>>>>>> >
>>>>> > >> >>>>>>>>>> > Thanks again
>>>>> > >> >>>>>>>>>> >
>>>>> > >> >>>>>>>>>> > Kind regards,
>>>>> > >> >>>>>>>>>> > Arjan Tijms
>>>>> > >> >>>>>>>>>> >
>>>>> > >> >>>>>>>>>> >
>>>>> > >> >>>>>>>>>> > On Thu, Mar 17, 2016 at 11:11 AM, Mark Thomas
>>>>> > >> >>>>>>>>>> <markt_at_apache.org <mailto:markt_at_apache.org>
>>>>> > >> >>>>>>>>>> <mailto:markt_at_apache.org <mailto:markt_at_apache.org
>>>>> >>
>>>>> > >> >>>>>>>>>> > <mailto:markt_at_apache.org
>>>>> > <mailto:markt_at_apache.org> <mailto:markt_at_apache.org
>>>>> > <mailto:markt_at_apache.org>>>>
>>>>> > >> wrote:
>>>>> > >> >>>>>>>>>> >
>>>>> > >> >>>>>>>>>> > On 07/03/2016 22:32, arjan tijms wrote:
>>>>> > >> >>>>>>>>>> > > Hi,
>>>>> > >> >>>>>>>>>> > >
>>>>> > >> >>>>>>>>>> > > On Mon, Mar 7, 2016 at 10:03 PM, Mark
>>>>> Thomas
>>>>> > >> >>>>>>>>>> <markt_at_apache.org <mailto:markt_at_apache.org>
>>>>> > <mailto:markt_at_apache.org <mailto:markt_at_apache.org>>
>>>>> > >> >>>>>>>>>> <mailto:markt_at_apache.org <mailto:markt_at_apache.org>
>>>>> > >> >>>>>>>>>> <mailto:markt_at_apache.org <mailto:
>>>>> markt_at_apache.org>>>
>>>>> > >> >>>>>>>>>> > > <mailto:markt_at_apache.org
>>>>> > <mailto:markt_at_apache.org>
>>>>> > ><mailto:markt_at_apache.org <mailto:markt_at_apache.org>>
>>>>> > >> >>>>>>>>>> <mailto:markt_at_apache.org <mailto:
>>>>> markt_at_apache.org>
>>>>> > <mailto:markt_at_apache.org <mailto:markt_at_apache.org>>>>>
>>>>> > >> wrote:
>>>>> > >> >>>>>>>>>> > >
>>>>> > >> >>>>>>>>>> > > I can look at implementing this in
>>>>> Tomcat
>>>>> > >if
>>>>> > >> that
>>>>> > >> >>>>>>>>>> would
>>>>> > >> >>>>>>>>>> be any use.
>>>>> > >> >>>>>>>>>> > >
>>>>> > >> >>>>>>>>>> > >
>>>>> > >> >>>>>>>>>> > > That would be a great step for sure.
>>>>> > >> >>>>>>>>>> > >
>>>>> > >> >>>>>>>>>> > > For me personally using it for Mojarra
>>>>> is a
>>>>> > >big use
>>>>> > >> >>>>>>>>>> case,
>>>>> > >> >>>>>>>>>> and its tests
>>>>> > >> >>>>>>>>>> > > run primarily against GlassFish, but I
>>>>> could
>>>>> > >sure
>>>>> > >> >>>>>>>>>> build a
>>>>> > >> >>>>>>>>>> personal test
>>>>> > >> >>>>>>>>>> > > bed based on Tomcat (in fact, we
>>>>> already have
>>>>> > >this
>>>>> > >> >>>>>>>>>> for
>>>>> > >> >>>>>>>>>> OmniFaces ;)).
>>>>> > >> >>>>>>>>>> > >
>>>>> > >> >>>>>>>>>> > > So if you can look at this that would
>>>>> be great
>>>>> > >:)
>>>>> > >> >>>>>>>>>> >
>>>>> > >> >>>>>>>>>> > I've implemented this in 9.0.0.M4
>>>>> available
>>>>> > >here:
>>>>> > >> >>>>>>>>>> > http://tomcat.apache.org/download-90.cgi
>>>>> > >> >>>>>>>>>> >
>>>>> > >> >>>>>>>>>> > As always, feedback appreciated.
>>>>> > >> >>>>>>>>>> >
>>>>> > >> >>>>>>>>>> > Mark
>>>>> > >> >>>>>>>>>> >
>>>>> > >> >>>>>>>>>> >
>>>>> > >> >>>>>>>>>>
>>>>> > >> >>>>>>>>>>
>>>>> > >> >>>>>>>>>
>>>>> > >> >>>>>>>>
>>>>> > >> >>>>>>
>>>>> > >> >>>>>>
>>>>> > >> >>>>>>
>>>>> > >> >>>>>>
>>>>> > >> >>>>>> --
>>>>> > >> >>>>>> Greg Wilkins <gregw_at_webtide.com
>>>>> > <mailto:gregw@webtide.com>> CTO http://webtide.com
>>>>> > >> >>>>>
>>>>> > >> >>>>>
>>>>> > >> >>>>
>>>>> > >> >>>>
>>>>> > >> >>>>
>>>>> > >> >>>> --
>>>>> > >> >>>> Greg Wilkins <gregw_at_webtide.com <mailto:
>>>>> gregw_at_webtide.com>>
>>>>> > CTO http://webtide.com
>>>>> > >> >>>
>>>>> > >> >>>
>>>>> > >> >>
>>>>> > >>
>>>>> > >>
>>>>> >
>>>>> >
>>>>> >
>>>>>
>>>>>
>>>>
>>>
>
>
> --
> Greg Wilkins <gregw@webtide.com> CTO http://webtide.com
>