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.

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

jcd.properties property not honored by MBeanSender #1

Open
arosien opened this issue Nov 9, 2009 · 2 comments
Open

jcd.properties property not honored by MBeanSender #1

arosien opened this issue Nov 9, 2009 · 2 comments

Comments

@arosien
Copy link

arosien commented Nov 9, 2009

The README says the you can add the jcd.* properties to a file specified by the property jcd.properties, but this actually doesn't work. The current implementation uses a static initializer in the Network class, but the properties loaded there are not referenced by the MBeanServer class.

@dougm
Copy link
Collaborator

dougm commented Nov 11, 2009

Hi Adam,
Thanks for the report. This should fix (untested), can you give this branch a try and I'll merge to master if its all good: http://github.com/hyperic/jcollectd/tree/ISSUE-1

@arosien
Copy link
Author

arosien commented Nov 13, 2009

I'll test it out. I'll probably refactor it a bit so that configuration is separate from the Network class (which is mainly just constants); it's a bit strange that that's where the global config is.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants