Uploaded image for project: 'CDI Specification Issues'
  1. CDI Specification Issues
  2. CDI-504

have a standard CDI annotation like @ConfigProperty from deltapsike


    • Type: Feature Request
    • Status: Open (View Workflow)
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:


      Docker and containerisation is the new hotness; in the docker world images are static and the main way of injecting configuration is via environment variables - so that the same image can be reused but configured at run time with different configuration values. (Changing configuration files requires either different images to be created or using custom volumes which is less ideal in a containerized world).

      So IMHO CDI should provide a simple, natural way to allow environment variables (or system properties) to be easily injected via CDI.

      Delta Spike as a @ConfigProperty annotation which works very nicely

      you can specify a name, default value and use it on an @Inject to provide a value from env vars, system properties or provide a default.

      Given how core and useful this is - it feels like it should be part of the CDI specification; its one simple annotation.

      Currently deltaspike provides a way to configure different sources and whatnot which is cool; I'd be happy if CDI just had a simpler annotation which was only bound to env vars / system properties.

      e.g. @EnvironmentProperty(name = "envvar", deafultValue="cheese")

      There could be debate on whether env vars or system properties should win if the same name is used for both; in a dockerized world I'd prefer env vars to win but folks can always tinker with their run command to make sure they pass in env vars as system properties to work around this I guess.

        Gliffy Diagrams




              • Assignee:
                jastrachan James Strachan
              • Votes:
                0 Vote for this issue
                10 Start watching this issue


                • Created: