[forge-dev] REST Plugin

[Dan Allen]

Ah. Thanks for pointing that out and a belated thanks for adding that
feature :)

I agree then, both fixes mentioned above should be pursued.

-Dan

--
Sent from my CyanogenMod-powered
Android device, an open platform for
carriers, developers and consumers.
On Aug 10, 2012 2:41 AM, "Paul Bakker" <paul.bakker.nl at gmail.com> wrote:

> Hi Dan,
>
> I have added support for @ApplicationPath a long time ago already :-)
> The user have the choice between using @ApplicationPath or a servlet
> mapping. We should support both, because @ApplicationPath only works in
> real application server. Although I think everybody should use that, Forge
> shouldn't assume so.
>
> The two points mentioned by Luca are still very valid points though.
>
> Paul
>
>
> On Aug 10, 2012, at 1:49 , Dan Allen <dan.j.allen at gmail.com> wrote:
>
> On Thu, Aug 9, 2012 at 3:32 PM, Luca Masini <luca.masini at gmail.com> wrote:
>
>> Hi guys, I've found two things that are not standard jax-rs 1.1
>>
>> 1) web.xml: it registers only the servlet-mapping (2.3.2 of the specs),
>> and this is not allowed
>>
>
> Personally I think the plugin should be using the annotation
> @ApplicationPath instead of the web.xml entry. I know we've debated this
> before, but this approach brings it inline with what we do in JDF and it's
> really the recommended way of activating JAX-RS IMO.
>
> @ApplicationPath("/rest")
> public class JaxRsActivator extends Application {
> }
>
> (The path should be selected based on what the user specifies during
> setup).
>
> -Dan
>
> --
> Dan Allen
> Principal Software Engineer, Red Hat | Author of Seam in Action
> Registered Linux User #231597
>
> http://google.com/profiles/dan.j.allen
> http://mojavelinux.com
> http://mojavelinux.com/seaminaction
>
> _______________________________________________
> forge-dev mailing list
> forge-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/forge-dev
>
>
>
> _______________________________________________
> forge-dev mailing list
> forge-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/forge-dev
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/forge-dev/attachments/20120810/5aafa199/attachment.html