-
Notifications
You must be signed in to change notification settings - Fork 36
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
Fingerprint documentation confusing #28
Comments
Copying @jakelandis' comment from elastic/logstash#7493 For all methods but There is an outstanding bug : #18 that should address the need to NOT use keys for simple (keyless) hashing. The key is really only important if you are trying to provide a means of nonrepudiation which (imo) is only relevant w/r/t security. If you are only using the fingerprint for uniquness, I would still suggest using one of the hashing functions, such as Example with multiple fields:
|
I believe that the documentation should be better now that https://github.com/logstash-plugins/logstash-filter-fingerprint/pull/37/files (the fix for #18) has been merged. It may take a little bit for the documentation to get published to elastic.co, but can be seen here in raw form: https://github.com/logstash-plugins/logstash-filter-fingerprint/blob/master/docs/index.asciidoc This will be released in |
The documentation for the fingerprint filter is very confusing.
See this issue for more details.
The text was updated successfully, but these errors were encountered: