<p>Ah. Thanks for pointing that out and a belated thanks for adding that feature :)</p>
<p>I agree then, both fixes mentioned above should be pursued.</p>
<p>-Dan</p>
<p>--<br>
Sent from my CyanogenMod-powered<br>
Android device, an open platform for<br>
carriers, developers and consumers.</p>
<div class="gmail_quote">On Aug 10, 2012 2:41 AM, "Paul Bakker" <<a href="mailto:paul.bakker.nl@gmail.com">paul.bakker.nl@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word">Hi Dan,<div><br></div><div>I have added support for @ApplicationPath a long time ago already :-)</div><div>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.</div>
<div><br></div><div>The two points mentioned by Luca are still very valid points though.</div><div><br></div><div>Paul</div><div><br></div><div><br><div><div>On Aug 10, 2012, at 1:49 , Dan Allen <<a href="mailto:dan.j.allen@gmail.com" target="_blank">dan.j.allen@gmail.com</a>> wrote:</div>
<br><blockquote type="cite">On Thu, Aug 9, 2012 at 3:32 PM, Luca Masini <span dir="ltr"><<a href="mailto:luca.masini@gmail.com" target="_blank">luca.masini@gmail.com</a>></span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi guys, I've found two things that are not standard jax-rs 1.1<div><br></div><div>1) web.xml: it registers only the servlet-mapping (2.3.2 of the specs), and this is not allowed</div></blockquote><div><br></div><div>
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.</div>
<div><br></div><div><div>@ApplicationPath("/rest")</div><div>public class JaxRsActivator extends Application {</div><div>}</div></div><div><br></div><div>(The path should be selected based on what the user specifies during setup).</div>
<div><br></div><div>-Dan</div><div><br></div></div>-- <br><div>Dan Allen</div>Principal Software Engineer, Red Hat | Author of Seam in Action<br>Registered Linux User #231597<br><br><div><a href="http://google.com/profiles/dan.j.allen" target="_blank">http://google.com/profiles/dan.j.allen</a><br>
<a href="http://mojavelinux.com/" target="_blank">http://mojavelinux.com</a><br><a href="http://mojavelinux.com/seaminaction" target="_blank">http://mojavelinux.com/seaminaction</a><br></div><br>
_______________________________________________<br>forge-dev mailing list<br><a href="mailto:forge-dev@lists.jboss.org" target="_blank">forge-dev@lists.jboss.org</a><br><a href="https://lists.jboss.org/mailman/listinfo/forge-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-dev</a><br>
</blockquote></div><br></div></div><br>_______________________________________________<br>
forge-dev mailing list<br>
<a href="mailto:forge-dev@lists.jboss.org">forge-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/forge-dev" target="_blank">https://lists.jboss.org/mailman/listinfo/forge-dev</a><br>
<br></blockquote></div>