Ok, it's quite a fast fix. I'll do it asap.<div>Ciao</div><div>L<span></span><br><br>Il giorno venerd́ 10 agosto 2012, Lincoln Baxter, III ha scritto:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hey Luca!<br><br>By all means submit a JIRA Issue and send a pull request! We'd be *very* happy about that!<br><br>~Lincoln<br><br><div class="gmail_quote">On Fri, Aug 10, 2012 at 3:18 AM, Luca Masini <span dir="ltr"><<a href="javascript:_e({}, 'cvml', 'luca.masini.mailing.list@gmail.com');" target="_blank">luca.masini.mailing.list@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thank you Paul.<div><br></div><div>Both are taken from the specs but while adding 3 lines to web.xml is not a huge task, the fact that WebLogic and TomEE doesn't desploy RESTful Services implemented by Stateful is blocking for me.</div>
<div><br></div><div><div><div><br><br><div class="gmail_quote">2012/8/10 Paul Bakker <span dir="ltr"><<a href="javascript:_e({}, 'cvml', 'paul.bakker.nl@gmail.com');" target="_blank">paul.bakker.nl@gmail.com</a>></span><br>
<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><div><div>On Aug 10, 2012, at 1:49 , Dan Allen <<a href="javascript:_e({}, 'cvml', 'dan.j.allen@gmail.com');" target="_blank">dan.j.allen@gmail.com</a>> wrote:</div>
<br></div></div><blockquote type="cite"><div><div>On Thu, Aug 9, 2012 at 3:32 PM, Luca Masini <span dir="ltr"><<a href="javascript:_e({}, 'cvml', '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></div></div>
_______________________________________________<br>forge-dev mailing list<br><a href="javascript:_e({}, 'cvml', '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="javascript:_e({}, 'cvml', '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>
<br></blockquote></div><br><br clear="all"><div><br></div></div></div><div>-- <br>****************************************<br><a href="http://www.lucamasini.net" target="_blank">http://www.lucamasini.net</a><br>
<a href="http://twitter.com/lmasini" target="_blank">http://twitter.com/lmasini</a><br>
<a href="http://www.linkedin.com/pub/luca-masini/7/10/2b9" target="_blank">http://www.linkedin.com/pub/luca-masini/7/10/2b9</a><br>****************************************<br>
</div></div>
<br>_______________________________________________<br>
forge-dev mailing list<br>
<a href="javascript:_e({}, 'cvml', '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>
<br></blockquote></div><br><br clear="all"><br>-- <br>Lincoln Baxter, III<br><a href="http://ocpsoft.org" target="_blank">http://ocpsoft.org</a><br>"Simpler is better."<br>
</blockquote></div><br><br>-- <br>****************************************<br><a href="http://www.lucamasini.net">http://www.lucamasini.net</a><br><a href="http://twitter.com/lmasini">http://twitter.com/lmasini</a><br><a href="http://www.linkedin.com/pub/luca-masini/7/10/2b9">http://www.linkedin.com/pub/luca-masini/7/10/2b9</a><br>
****************************************<br>