74

I'm trying to inject a Stateless EJB into my JAX-RS webservice via annotations. Unfortunately the EJB is just null and I get a NullPointerException when I try to use it.

@Path("book")
public class BookResource {

    @EJB
    private BookEJB bookEJB;

    public BookResource() {
    }

    @GET
    @Produces("application/xml")
    @Path("/{bookId}")
    public Book getBookById(@PathParam("bookId") Integer id)
    {
        return bookEJB.findById(id);
    }
}

What am I doing wrong?

Here is some information about my machine:

  • Glassfish 3.1
  • Netbeans 6.9 RC 2
  • Java EE 6

Can you guys show some working example?

0

7 Answers 7

115

I am not sure this is supposed to work. So either:

Option 1: Use the injection provider SPI

Implement a provider that will do the lookup and inject the EJB. See:

Example for com.sun.jersey:jersey-server:1.17 :

import com.sun.jersey.core.spi.component.ComponentContext;
import com.sun.jersey.core.spi.component.ComponentScope;
import com.sun.jersey.spi.inject.Injectable;
import com.sun.jersey.spi.inject.InjectableProvider;

import javax.ejb.EJB;
import javax.naming.Context;
import javax.naming.InitialContext;
import javax.ws.rs.ext.Provider;
import java.lang.reflect.Type;

/**
 * JAX-RS EJB Injection provider.
 */
@Provider
public class EJBProvider implements InjectableProvider<EJB, Type> {

    public ComponentScope getScope() {
        return ComponentScope.Singleton;
    }

    public Injectable getInjectable(ComponentContext cc, EJB ejb, Type t) {
        if (!(t instanceof Class)) return null;

        try {
            Class c = (Class)t;
            Context ic = new InitialContext();

            final Object o = ic.lookup(c.getName());

            return new Injectable<Object>() {
                public Object getValue() {
                    return o;
                }
            };
        } catch (Exception e) {
            e.printStackTrace();
            return null;
        }
    }
}

Option 2: Make the BookResource an EJB

@Stateless
@Path("book")
public class BookResource {

    @EJB
    private BookEJB bookEJB;

    //...
}

See:

Option 3: Use CDI

@Path("book")
@RequestScoped
public class BookResource {

    @Inject
    private BookEJB bookEJB;

    //...
}

See:

8
  • Dear @Pascal Thivent Is that making JAX-Resources as @Stateless the only way to go?
    – Jin Kwon
    Jun 8, 2012 at 9:05
  • @Pascal Option 3 can also look like this: @EJB private BookEJB bookEJB and it will work (GF 3.1.2)
    – NBW
    May 6, 2013 at 2:17
  • 2
    "I am not sure this is supposed to work." - Why? I would expect you could just inject an EJB without playing a bunch of games
    – Ryan
    Sep 24, 2014 at 16:47
  • @Ryan: no it's not easy at all because of Glassfish 4.1 HK2 bugs. I faced the same issue today. Feb 20, 2015 at 6:13
  • @Pascal Thivent: Are there any disadvantages of 2nd option? like.. it will increase beans in jboss?
    – nilesh
    May 13, 2015 at 5:58
15

This thread is rather old, nevertheless i fought the same problem just yesterday. Here is my solution:

Just make the BookResource a managed bean through @javax.annotation.ManagedBean at class level.

For this to work you need to enable CDI with a beans.xml:

<?xml version="1.0" encoding="UTF-8"?>
<beans xmlns="http://java.sun.com/xml/ns/javaee"
       xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
       xsi:schemaLocation="http://java.sun.com/xml/ns/javaee http://java.sun.com/xml/ns/javaee/beans_1_0.xsd">
</beans>

This file needs to be in WEB-INF if the BookResource is part of a war file. If the BookResource is packaged with the ejbs put it into META-INF.

If you want to use @EJB you're done. If you want to inject the EJB through @Inject than a beans.xml must be put into the ejbs jar file into META-INF as well.

What you're doing: You're just telling the container that the resource should be container managed. Therefor it supports injection as well as lifecycle events. So you have your business facade without promoting it to an EJB.

You don't need to extend javax.ws.rs.core.Application for this to work. BookResource is as a root resource automatically request scoped.

Tested with Glassfish 3.1.2 and a maven project.

Happy coding.

2
  • I know this answer is extremely old but: META-INF/beans.xml doesn't have anything to do with the Java EE @ManagedBean annotation; you shouldn't need it. I too use @ManagedBean to get my resource class to permit @EJB references inside it and didn't need META-INF/beans.xml involved. It's worth noting as well that if you do this you are hosed if you want to override the @EJB annotations later in a deployment descriptor, because there isn't one for @ManagedBeans. Jun 25, 2013 at 20:29
  • I had the same problem. I needed beans.xml to get it running. My environment was JBoss EAP 6.4 with jersey rest service. I ran into JBAS011859. But at the end it was running. Apr 12, 2016 at 8:00
11

You shall be able to do injection in JAX-RS resource without making it EJB or CDI component. But you have to remember that your JAX-RS resource must not be singleton.

So, you setup your application with this code. This makes BookResource class per-request JAX-RS resource.

@javax.ws.rs.ApplicationPath("application")
public class InjectionApplication extends javax.ws.rs.core.Application {
  private Set<Object> singletons = new HashSet<Object>();
  private Set<Class<?>> classes = new HashSet<Class<?>>();

  public InjectionApplication() {
    // no instance is created, just class is listed
    classes.add(BookResource.class);
  }

  @Override
  public Set<Class<?>> getClasses() {
    return classes;
  }

  @Override
  public Set<Object> getSingletons() {
    return singletons;
  }
}

With this setup, you are letting JAX-RS to instantiate BookResource for you on per-request basis and also inject all the required dependencies. If you make BookResource class singleton JAX-RS resource, this is, you put in getSingletons

public Set<Object> getSingletons() {
  singletons.add(new BookResource());
  return singletons;
}

then, you created instance which is not managed by JAX-RS runtime and nobody in container cares to inject anything.

3
  • The 'singleton' thing is not relevant with @EJB problem, right?
    – Jin Kwon
    Jun 8, 2012 at 9:49
  • 1
    In my case, my REST resource was singleton. So even if I inject session bean (using @EJB or @Inject), by bean instance was always null, causing NullPointerException. Your last hint regarding singleton, helped me to resolve this issue. I made sure that my REST resource is not singleton.
    – user613114
    May 30, 2014 at 13:09
  • Old post but thank you so much @Martin! I was struggeling with this problem since yesterday. Your hint with singleton made it.
    – peez80
    May 3, 2015 at 12:20
5

Unfortunately, my answer is too long for a comment, so here goes. :)

Zeck, I hope that you are aware of what exactly you are doing by promoting your bean to an EJB, as suggested by Pascal. Unfortunately, as easy as it is nowadays with Java EE to 'make a class an EJB', you should be aware of the implications of doing so. Each EJB creates overhead along with the additional functionality it provides: they are transaction aware, have their own contexts, they take part in the full EJB life cycle, etc.

What I think you should be doing for a clean and reusable approach is this: extract the access to your servers services (which hopefully are accessed through a SessionFacade :) into a BusinessDelegate. This delegate should be using some kind of JNDI lookup (probably a ServiceLocator - yes, they are still valid in Java EE!) to access your backend.

Okay, off the record: if you really, really, really need the injection because you do not want to write JNDI access manually, you could still make your delegate an EJB, although it ... well, it just feels wrong. :) That way at least it will be easy to replace it later with something else if you do decide to switch to a JNDI lookup approach...

6
  • Errr, that is what you get when you do not read the question completely. In the case of your RESTful service, arguably, the service itself is your BusinessDelegate. So there you have it: the first and last part of my statement are still totally valid, while I would probably not create a BusinessDelegate for the BusinessDelegate... :)
    – LeChe
    Sep 1, 2011 at 7:20
  • I wasn't comfortable making my api endpoints EJBs neither, but you can just use @ManagedBean. See my answer, if you like Oct 12, 2012 at 8:10
  • Your offer of JNDI is developed in Option 1: Use the injection provider SPI example. Feb 27, 2013 at 5:52
  • 2
    People often talk about EJB "overhead" as if it were a well-established fact that they are heavyweight, but most developers have never bothered to measure it. Adam Bien showed that the performance difference between EJBs and POJOs was small (< 10%) in 2008 (adam-bien.com/roller/abien/entry/is_it_worth_using_pojos), and performance has only improved since then. In short, if you're trying to optimize your application for performance by comparing POJOs to EJBs, you're looking in the wrong place.
    – user201891
    Feb 15, 2017 at 23:11
  • 1
    I think you're misinterpreting that 10% number, LeChe. It's not "my application is now 10% slower", it's one specific type of operation in the application that is 10% slower. Using EJBs instead of POJOs doesn't make network, DB, file IO, and rendering take 10% more time, and these operations make up the bulk of many systems. The overall performance impact of using EJBs versus POJOs on a real application is probably negligible. As for the latter part of your comment, I agree there are good reasons to not make everything an EJB: fortunately Pascal suggested two alternatives.
    – user201891
    Feb 16, 2017 at 20:18
3

I was trying to do the exact same thing. I'm using EJB 3.1 and have a deployed my app as an EAR with separate EJB project. As Jav_Rock pointed out, I use context lookup.

@Path("book")
public class BookResource {

  @EJB
  BookEJB bookEJB;

  public BookResource() {
    try {
        String lookupName = "java:global/my_app/my_ejb_module/BookEJB";
        bookEJB = (BookEJB) InitialContext.doLookup(lookupName);
    } catch (NamingException e) {
        e.printStackTrace();
    }
  }

  @GET
  @Produces("application/xml")
  @Path("/{bookId}")
  public Book getBookById(@PathParam("bookId") Integer id) {
    return bookEJB.findById(id);
  }
}

See the link below for very useful JNDI look up tips

JNDI look up tips

2
  • I did use this method and it did work.. However, the same code works in jboss well... So I am really not sure why this happens..
    – mwangi
    Jul 4, 2015 at 19:56
  • I had to use this for an legacy app. yet in jboss 6.4 i found that only the complete jndi name worked: java:global/completeEar/completeEar-ejb-1.0-SNAPSHOT/TestEJBImpl!personal.ejb.TestEJBRemote
    – cabaji99
    Oct 11, 2017 at 19:47
2

Arjan is right. I created another class to initialize the EJB instead of creating a bean for RS

@Singleton
@LocalBean
public class Mediator {
    @EJB
    DatabaseInterface databaseFacade;

to avoid null pointer with:

@Path("stock")
public class StockResource {
    @EJB
    DatabaseInterface databaseFacade;
...

it actually works on GF

0

I have the same problem, and I solved it calling te EJB by a context lookup (the injection was impossible, I had the same error NullPointerException).

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge you have read our privacy policy.

Not the answer you're looking for? Browse other questions tagged or ask your own question.