Pierre De Rop created FELIX-3700:
------------------------------------

             Summary: DS Factory Components don't support configuration-policy 
= require
                 Key: FELIX-3700
                 URL: https://issues.apache.org/jira/browse/FELIX-3700
             Project: Felix
          Issue Type: Bug
          Components: Declarative Services (SCR)
         Environment: linux, jdk1.6
            Reporter: Pierre De Rop


With scr 1.6.0 and also with scr from trunk, there is the following problem:

When a factory component (declared with a factory attribute in the Component 
element), and when the Component is also defined with configuration-policy = 
require, then the org.osgi.service.component.ComponentFactory associated to the 
factory component is registered in the OSGi registry even if the configuration 
for the component is not yet available.

This is  a problem because when the org.osgi.service.component.ComponentFactory 
is registered in the registry, then another component using the 
ComponentFactory may call the newInstance method and then instantiate the 
component without the required configuration.

For example, in the following code, Main is injected with the A 
ComponentFactory and creates immediately one A instance: but at this point, the 
Configuration for A has not yet been created (see in the Main.start method, 
where a thread is started in order to create the A configuration after 1 second 
...).

I expect A ComponentFactory to be registered only after the A Configuration is 
available from config admin, because the A configuration-policy has been set to 
the "require" value.

Am I correct or does the spec forbid factory components to use the 
configuration-policy = require ?

->

@Component(factory = "A", configurationPolicy = ConfigurationPolicy.require)
public class A {
  @Activate
  void start(Map<?, ?> config) {
    System.out.println("A.start:" + config.get("foo");
  }
}

@Component
public class Main {
  private ConfigurationAdmin _cm;
  
  @Reference(type = '*', target = "(component.factory=A)")
  void bindCF(ComponentFactory cf) { // should only be called once A config 
admin configuration is avail
    System.out.println("Main.bindCF");
    cf.newInstance(null);
  }
  
  @Reference
  void bindCM(ConfigurationAdmin cm) {
    _cm = cm;
  }
  
  @Activate
  void start() {
    System.out.println("Main.start");
    
    new Thread(new Runnable() {
      public void run() {
        try {
          Thread.sleep(1000);
          System.out.println("Configuring A");          
          Configuration config = _cm.getConfiguration("A", null);
          config.update(new Hashtable() {
            {
              put("foo", "bar");
            }
          });          
        } catch (Exception ioe) {
          ioe.printStackTrace();
        }
      }
    }).start();
  }
}


--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to