Ok, it's quite a fast fix. I'll do it asap.
Ciao
L

Il giorno venerd́ 10 agosto 2012, Lincoln Baxter, III ha scritto:
Hey Luca!

By all means submit a JIRA Issue and send a pull request! We'd be *very* happy about that!

~Lincoln

On Fri, Aug 10, 2012 at 3:18 AM, Luca Masini <luca.masini.mailing.list@gmail.com> wrote:
Thank you Paul.

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.



2012/8/10 Paul Bakker <paul.bakker.nl@gmail.com>
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




--
****************************************
http://www.lucamasini.net
http://twitter.com/lmasini
http://www.linkedin.com/pub/luca-masini/7/10/2b9
****************************************

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




--
Lincoln Baxter, III
http://ocpsoft.org
"Simpler is better."


--
****************************************
http://www.lucamasini.net
http://twitter.com/lmasini
http://www.linkedin.com/pub/luca-masini/7/10/2b9
****************************************