Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

SE config should prioritize system properties and env vars as does MP #507

Closed
batsatt opened this issue Mar 14, 2019 · 1 comment
Closed

SE config should prioritize system properties and env vars as does MP #507

batsatt opened this issue Mar 14, 2019 · 1 comment
Assignees
@batsatt
Labels
2.x Issues for 2.x version branch config SE

Comments

@batsatt
Copy link
Contributor

batsatt commented Mar 14, 2019

Environment Details

  • Helidon Version: 1.0.0-SNAPSHOT
  • Helidon SE

Problem Description

MP prioritizes system properties (ordinal 400) over environment variables (ordinal 300), while SE does the reverse, so produces different merged results when a given key is present in both.
@batsatt batsatt added the config label Mar 14, 2019
@batsatt batsatt self-assigned this Mar 14, 2019
@batsatt batsatt mentioned this issue Mar 15, 2019
Merged
@m0mus m0mus added the SE label Apr 15, 2019
@batsatt batsatt mentioned this issue Sep 7, 2019
Closed
@tomas-langer
Copy link
Member

This should now be fixed, as Helidon SE Config implements MP Config as well.
Fixed for 2.0 only, as this is a backward incompatible chane.

@tomas-langer tomas-langer added the 2.x Issues for 2.x version branch label Jan 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@batsatt batsatt

Labels
2.x Issues for 2.x version branch config SE
Projects
Backlog
Archived in project
Milestone
No milestone
Development

No branches or pull requests
3 participants
@batsatt @m0mus @tomas-langer