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

SystemConfig getListProperty reads empty declared variable as List with one element #101

Open
garyluoex opened this issue Nov 18, 2016 · 2 comments
Open

SystemConfig getListProperty reads empty declared variable as List with one element #101

garyluoex opened this issue Nov 18, 2016 · 2 comments
Labels
BUG CONFIG

Comments

@garyluoex
Copy link
Collaborator

garyluoex commented Nov 18, 2016 •
edited

  • Found when using moduleConfig.properties with user_roles variable here while working on this PR
  • Might be an org.apache.commons.configuration.CompositeConfiguration bug
  • Here is getListProperty
@garyluoex garyluoex added the BUG label Nov 18, 2016
@cdeszaq
Copy link
Collaborator

cdeszaq commented Nov 19, 2016

Good catch! We'll try to get this fixed before the next stable release (ie. before we call v0.7.x stable)

@cdeszaq
Copy link
Collaborator

cdeszaq commented Nov 19, 2016

When we go to fix this, it may be worthwhile to tackle upgrading to the v2 line of the Apache Configuration. Based on the upgrade guide, it looks like list handling is one area that has gotten a fair bit of love and attention, as well as hierarchical configurations, which is what we've got going on.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
BUG CONFIG
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@cdeszaq @kevinhinterlong @garyluoex