-
Notifications
You must be signed in to change notification settings - Fork 97
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
JA4SSH in its own log. #91
Comments
@zrobinette12 great question. I'm open to thoughts on this. The reason was that JA4SSH will generate a new log line every 200 ssh packets. For immediate detection and response purposes, we did not want to wait until after the ssh session was closed to log the fingerprints. Instead, we're logging the fingerprints as they happen, in ja4ssh.log. Think of it like files.log or x509.log which log files and certs as they are seen rather than waiting for the sessions to close to log in http.log or ssl.log, respectively. That said, we could log in ssh.log if users want us to go that route. |
@john-althouse Ah I see, the old Zeek long connections conundrum. I’d have a use case for both, logging the fingerprint in the existing ssh.log and having the ja4ssh.log but unsure if others would. Mine is simply a budget issue where I can’t ingest new Zeek logs no matter how much we want immediate detection and response. Adding a single field to an existing log is much more doable. Perhaps a config option to also log the fingerprint in the ssh.log? |
@zrobinette12 did you find a solution or are you still looking for that config option? |
@john-althouse no solution yet unfortunately. Having a config option would be great for us but unsure if others would feel the same. |
Does a similar logic applies to Wireshark plugin ? Edit: hmm, it seems it works for some connection but not for all. I have a default SSH config without any hardening for which it works. |
Is there a reason behind the Zeek ssh.log not containing JA4SSH and having that data broken out into its own ja4ssh.log?
The text was updated successfully, but these errors were encountered: