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

[Feature Request] Jolokia: JMX-path should be configurable #2005

Closed
kingcrunch opened this issue Nov 7, 2016 · 1 comment
Closed

[Feature Request] Jolokia: JMX-path should be configurable #2005

kingcrunch opened this issue Nov 7, 2016 · 1 comment
Labels
help wanted Request for community participation, code, contribution
Milestone
1.5.0

Comments

@kingcrunch
Copy link

Feature Reqeust

With Jolokia in Proxy mode the JMX-Context is harcoded to /jmxrmi. Not every setup use this default.

Proposal:

Make JMX-path configurable

Current behavior:

Path not configurable. It is hardcoded top /jmxrmi

Desired behavior:

When using jolokia in proxy-mode it should be possible use different jmx-paths than /jmxrmi

Use case

Right now jolokia is not usable at all with setups, that use different jmx-path than /jmxrmi
@kingcrunch kingcrunch added this to the Future Milestone milestone Nov 7, 2016
@kingcrunch kingcrunch added the help wanted Request for community participation, code, contribution label Nov 7, 2016
@dylanmei dylanmei mentioned this issue Feb 8, 2017
Merged
8 tasks
@danielnelson danielnelson removed this from the Future Milestone milestone Jun 14, 2017
@danielnelson danielnelson added this to the 1.5.0 milestone Sep 27, 2017
@danielnelson
Copy link
Contributor

I believe this has been fixed in #2278 which will be included in 1.5.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted Request for community participation, code, contribution
Projects
None yet
Milestone
1.5.0
Development

No branches or pull requests
2 participants
@kingcrunch @danielnelson