-
Notifications
You must be signed in to change notification settings - Fork 163
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
feat: wazuh bf #1215
base: master
Are you sure you want to change the base?
feat: wazuh bf #1215
Conversation
Hey 👋🏻 Thank you for opening a pull request! There a few areas I see some improvement, we can use the To parse the json string into a defined json object, you can see Caddy parser on how you can also use it within this parser. Then once we unmarshal the json to the object we can then parse the note please redact any PII from the log lines such as IP address or User information, you can use place holders to reflect this information and we will fill it with fake data such as |
Can you take a look, what do you think about it? Does it seem better to you? |
Nearly there! Since we need to know if it "failing auth" we could do this: - meta: log_type
expression: |
evt.Unmarshaled.wazuh.type == 'response' && evt.Unmarshaled.wazuh.method == 'post' && evt.Unmarshaled.wazuh.statusCode in [401,'401'] && evt.Unmarshaled.wazuh.req.url == '/auth/login' ? 'wazuh_failed_auth' : '' Try it you may need to wrap all the |
like this ? successful test |
Maybe if we can add some tests we can ensure if it works or not, I can add those if you add them here. |
You may need two logs? One with a 200 code and another with a 401 code? |
|
Is it all good? Or do I need to revise the PR? |
Apologies been a manic week, I need to add the tests the only question is do we rename it from |
Very well. Regarding |
Apologies in the delay, could we rename all elements of I will wait for the changes, then proceed to add the tests. |
successful test