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

Cassandra metric on https #630

Closed
MGulerce opened this issue Aug 26, 2021 · 9 comments
Closed

Cassandra metric on https #630

MGulerce opened this issue Aug 26, 2021 · 9 comments

Comments

@MGulerce
Copy link

Hello,
I'm new to jmx-exporter.I setup the jar with standalone and agent .Works fine each way.But metric can read on http://IP:7070
But I cant add ssl encryption.I dont know if it's possible to do.
If it's possible how can ı do that?

Thanks for helping.

@tashian
Copy link

tashian commented Sep 1, 2021

see #442

@MGulerce
Copy link
Author

MGulerce commented Sep 1, 2021

Hi tashian thanks for comment and helping but as ı understand this line
"Just to be clear: The PR mentioned above adds basic HTTP authentication. It does not add SSL support." in #442
It supports http authentication not ssl
But ı will try to do that at least .
If you have any experiment whit this I'm waiting for any advice or direction.
Thanks again

@tashian
Copy link

tashian commented Sep 1, 2021

Sure, as far as I can tell this project doesn't support TLS. My suggestion would be to have the jmx exporter bind to localhost, and then run a reverse proxy in front of it (eg. Caddy or Traefik) that adds TLS server authentication. See https://caddyserver.com/docs/caddyfile/directives/reverse_proxy for more.

@MGulerce
Copy link
Author

MGulerce commented Sep 1, 2021

Thank you for helping
I will check it out and see what ı can do

@fstab
Copy link
Member

fstab commented Sep 15, 2021

@dhoard started working on it prometheus/client_java#686

@dhoard
Copy link
Collaborator

dhoard commented Sep 15, 2021

@MGulerce as @fstab has pointed out, I am in the process of adding cleaner SSL support into client_java

Issue -> prometheus/client_java#693
PR -> prometheus/client_java#695

If / when client_java is released with these changes, exporters can then be modified to support SSL and BASIC authentication.

In the short term, @tashian 's suggestion is the current approach.

@MGulerce MGulerce reopened this Sep 19, 2021
@MGulerce
Copy link
Author

Thank you,
We use our system as @tashian said , But It would be great to add SSL support on client_java .
So I reaopened the ıssue

@dhoard
Copy link
Collaborator

dhoard commented Sep 20, 2021

As pointed out by @tashian this a dupe of #442

I propose we close this issue and track SSL/TLS support in that issue.

@MGulerce
Copy link
Author

MGulerce commented Sep 20, 2021

Okey than
Thanks again for helping

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

4 participants