[CONFIGURATION-447] DefaultConfigurationBuilder should provide default provider for environment variables

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 1.6
• Fix Version/s: 1.7
• Assignee: Unassigned
• Reporter: Fabien Nisol
• Created: 18/Apr/11 15:00
• Updated: 08/Sep/11 09:20
• Resolved: 21/Apr/11 19:48

Description

DefaultConfigurationBuilder does not provide a way to merge environment variable into the configuration

eg:

<configuration>
  <env/>
  ...
</configuration>

I think this could be implemented by adding the following code into the DefaultConfigurationBuilder class

DefaultConfigurationBuilder.java

//Constant for the provider for environment variables.
private static final ConfigurationProvider ENV_PROVIDER = new ConfigurationProvider(
            EnvironmentConfiguration.class);

//An array with the names of the default tags.
 private static final String[] DEFAULT_TAGS =
     {"properties", "xml", "hierarchicalXml", "jndi", "system", "plist", "configuration","env"};

    
//An array with the providers for the default tags.
 
     private static final ConfigurationProvider[] DEFAULT_PROVIDERS =
     {PROPERTIES_PROVIDER, XML_PROVIDER, XML_PROVIDER, JNDI_PROVIDER,
             SYSTEM_PROVIDER, PLIST_PROVIDER, BUILDER_PROVIDER,ENV_PROVIDER};