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

--tls generates error, drops events #288

Open
tempire opened this issue Feb 5, 2024 · 3 comments
Open

--tls generates error, drops events #288

tempire opened this issue Feb 5, 2024 · 3 comments

Comments

@tempire
Copy link

tempire commented Feb 5, 2024

Issue

When the --tls switch is used:

riemann-health --host example.com --port 5554 --tcp --tls

No events are sent to the server.

Dropped 113 events due to no implicit conversion of nil into String

What's your riemann-tools version?

riemann-tools (1.10.0)

What platform are you on?

Same result on macos, alpine, and ami2 linux. Works fine when not sending to tls-enabled server.

Events work fine using tls in nodejs.

@smortex
Copy link
Member

smortex commented Feb 6, 2024

Hey! Using --tls without --tls-ca-cert, --tls-cert and --tls-key? I guess you can skip --tls-ca-cert if you decide to ignore the CA verification (--no-tls-verify), but key and cert MUST be provided for mutual-TLS authentication…

Maybe we should fail hard at startup if --tls is requested but some parameters are missing?

@tempire
Copy link
Author

tempire commented Feb 6, 2024 via email

@smortex
Copy link
Member

smortex commented Feb 6, 2024

I want secure communication, not authentication

Hum, is this supported by riemann? The server seems to assert presence of cert, key and ca-cert to start a TLS server:
https://github.com/riemann/riemann/blob/2f06db732bdf65108572e0a37212719a915b6c2c/src/riemann/transport/tcp.clj#L281-L284

It looks like riemann-client currently assume mutual-TLS authentication which seem aligned with this:
https://github.com/riemann/riemann-ruby-client/blob/main/lib/riemann/client/ssl_socket.rb#L20-L21

Events work fine using tls in nodejs.

Do you say you made unauthenticated TLS work with a nodejs client? Can you share your riemann config? My attempts to start an unauthenticated TLS server failed…

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