Weld
  1. Weld
  2. WELD-687

Refactoring InstantiatorFactory to allow per-deployment configuration

    Details

    • Similar Issues:
      Show 10 results 

      Description

      InstantiatorFactory is currently using static fields for storing the enabled/disabled status for unsafe instantiators, as well as the instantiator that should be used. In a shared library scenario, the instantiator factory will be enabled if at least one of the deployed applications contains the META-INF/org.jboss.weld.enableUnsafeProxies resource.

        Activity

        Hide
        Marius Bogoevici
        added a comment -

        Actually. it's more complicated than this. InstantiatorFactory will decide on whether unsafe proxies are enabled or not depending on whether the first CDI application deployed on the server had unsafe proxies enabled or not.

        Show
        Marius Bogoevici
        added a comment - Actually. it's more complicated than this. InstantiatorFactory will decide on whether unsafe proxies are enabled or not depending on whether the first CDI application deployed on the server had unsafe proxies enabled or not.
        Hide
        Marius Bogoevici
        added a comment -

        Marked as a bug, this should not work only when weld is shared.

        Show
        Marius Bogoevici
        added a comment - Marked as a bug, this should not work only when weld is shared.
        Hide
        Pete Muir
        added a comment -

        Disabled

        org.jboss.weld.tests.proxy.privateconstructor.ProxyConstructorTest
        org.jboss.weld.tests.proxy.weld56.ProxyTest
        org.jboss.weld.tests.proxy.weld56.ProxyTest

        until this issue is fixed.

        Show
        Pete Muir
        added a comment - Disabled org.jboss.weld.tests.proxy.privateconstructor.ProxyConstructorTest org.jboss.weld.tests.proxy.weld56.ProxyTest org.jboss.weld.tests.proxy.weld56.ProxyTest until this issue is fixed.
        Hide
        Pete Muir
        added a comment -

        We should alter InstantiatorFactory to be a per-BDA internal-only service.

        Show
        Pete Muir
        added a comment - We should alter InstantiatorFactory to be a per-BDA internal-only service.
        Hide
        Pete Muir
        added a comment -

        Nik, can you take a look at this?

        Show
        Pete Muir
        added a comment - Nik, can you take a look at this?
        Hide
        Pete Muir
        added a comment -

        Nik, any hope you can look at this?

        Show
        Pete Muir
        added a comment - Nik, any hope you can look at this?

          People

          • Assignee:
            Ales Justin
            Reporter:
            Marius Bogoevici
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: