-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Filebeat][HTTPJson Input]fixes issues with mapstring against JSONObject config #19069
Conversation
Pinging @elastic/siem (Team:SIEM) |
Would be possible to add a small test for it? Other than that LGTM 👍 |
Done, ran all tests which passed, the new test would look like this, also tested with modifying it to fail, which it also did if it was incorrect:
|
Relates: #18919 (comment) This will partially address my reported issue (pagination is still a problem for the scroll example). |
…ect config (elastic#19069) * fixes issues with mapstring against JSONobject, mm is always nil * added test for array response working (cherry picked from commit 6d91d9f)
…ect config (elastic#19069) * fixes issues with mapstring against JSONobject, mm is always nil * added test for array response working (cherry picked from commit 6d91d9f)
…ect config (elastic#19069) * fixes issues with mapstring against JSONobject, mm is always nil * added test for array response working (cherry picked from commit 6d91d9f)
…ect config (elastic#19069) * fixes issues with mapstring against JSONobject, mm is always nil * added test for array response working
…ect config (elastic#19069) (elastic#19075) * fixes issues with mapstring against JSONobject, mm is always nil * added test for array response working (cherry picked from commit d1db38e) Co-authored-by: Marius Iversen <pillus@chasenet.org>
What does this PR do?
This change resolves an error when using "json_objects_array" configuration option. Currently the feature does not work due to mm always being nil
Why is it important?
Fixes a currently unusable feature
Checklist
- [ ] I have added an entry inCHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.