[forge-dev] REST Plugin

Paul Bakker paul.bakker.nl at gmail.com
Fri Aug 10 02:41:30 EDT 2012


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

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/forge-dev/attachments/20120810/5546d329/attachment.html 


More information about the forge-dev mailing list