Weld
  1. Weld
  2. WELD-570

Partial export of javasisst from Weld breaks other javasisst clients

    Details

    • Steps to Reproduce:
      Hide

      A small sample webapp exhibiting the problem is attached to https://glassfish.dev.java.net/issues/show_bug.cgi?id=12368.

      Show
      A small sample webapp exhibiting the problem is attached to https://glassfish.dev.java.net/issues/show_bug.cgi?id=12368 .
    • Similar Issues:
      Show 10 results 

      Description

      weld-osgi-bundle.jar (as used in Glassfish 3.0 and higher) still contains a buggy version of javasisst causing memory leaks.

      See WELD-453 and JASSIST-104. Even though Weld itself no longer uses the problematic javassist ProxyFactory, it is still the root cause for a memory leak occurring when HIbernate (or potentially any other Javassist client) is used on Glassfish.

      The problem exists in Glassfish 3.0.1 and also in the promoted build 3.1-b12 using a 1.1.0 prerelease of Weld, see https://glassfish.dev.java.net/issues/show_bug.cgi?id=12368

      weld-osgi-bundle.jar contains a version of javassist which is almost but not fully private. There is a package export for javassist.util.proxy. When deploying a WAR on Glassfish containing Hibernate (3.5.3) and another copy of Javasisst required by Hibernate, then by parent classloader delegation Hibernate will end up using the javassist ProxyFactory from Weld, registering its proxies deep down in the container, and thus causing a memory leak with each redeployment of the WAR.

      Weld should either keep its included version of javassist fully private or import all of javasisst via Import-Package, relying on javasisst installed in a separate bundle.

        Issue Links

          Activity

          Hide
          Pete Muir
          added a comment -

          This was fixed in Beta2

          Show
          Pete Muir
          added a comment - This was fixed in Beta2

            People

            • Assignee:
              Roger Kitain
              Reporter:
              Harald Wellmann
            • Votes:
              1 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: