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@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@gmail.com> wrote:

On Thu, Aug 9, 2012 at 3:32 PM, Luca Masini <luca.masini@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


_______________________________________________
forge-dev mailing list
forge-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/forge-dev


_______________________________________________
forge-dev mailing list
forge-dev@lists.jboss.org
https://lists.jboss.org/mailman/listinfo/forge-dev