Uploaded image for project: 'Seam Transaction'
  1. Seam Transaction
  2. SEAMTRANSACTION-15

Seam Transaction is no longer working with servlet containers like Tomcat that don't support CDI injection in listeners

    Details

      Description

      Lack of transaction injection causes:

      java.lang.NullPointerException
      org.jboss.seam.transaction.TransactionServletListener.requestInitialized(TransactionServletListener.java:106)
      org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:98)
      org.apache.catalina.valves.AccessLogValve.invoke(AccessLogValve.java:927)
      org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:407)
      org.apache.coyote.http11.AbstractHttp11Processor.process(AbstractHttp11Processor.java:987)
      org.apache.coyote.AbstractProtocol$AbstractConnectionHandler.process(AbstractProtocol.java:579)
      org.apache.tomcat.util.net.JIoEndpoint$SocketProcessor.run(JIoEndpoint.java:307)
      java.util.concurrent.ThreadPoolExecutor$Worker.runTask(ThreadPoolExecutor.java:886)
      java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:908)
      java.lang.Thread.run(Thread.java:662)

      The implementation has changed after SEAM Transaction 3.1.0.Beta5:

      https://github.com/seam/transaction/commit/64ef15aee882d8809a42749fc2283a1dd53542be

      Instead of a filter there is now a listener doing the job. This "better" implementation excludes all users that use non-EE-containers like Tomcat that don't support @Inject in listeners:

      e.g. Tomcat 7.0.25 startup:

      ...
      INFO: Tomcat 7 detected, CDI injection will be available in Servlets and Filters
      . Injection into Listeners is not supported
      ...

        Gliffy Diagrams

          Attachments

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                miguelzp Miguel Z
              • Votes:
                4 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated: