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.

Sign up for GitHub

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

Jump to bottom

[7.x] fix 400 error on initial signals search (#70618) #70662

Merged
merged 1 commit into from
Jul 3, 2020
Merged

[7.x] fix 400 error on initial signals search (#70618) #70662

merged 1 commit into from
Jul 3, 2020

Commits on Jul 2, 2020

  1. fix 400 error on initial signals search (elastic#70618) 

    ### Summary

On initial render of the SIEM pages, a 400 error was showing for POST http://localhost:5601/api/detection_engine/signals/search. This PR is a temporary fix for this bug. This initial call is being used to populate the Last alert text that shows at the top of a number of the pages. The reason the size was 0 is because we weren't interested in the signals themselves, just the timestamp of the last alert. Teamed up with @XavierM and it seems to us that the issue is the server side validation. It may be Hapi misreading the 0 as false or our updated validation not accepting size 0.
    @yctercero
    yctercero committed Jul 2, 2020
    Configuration menu
    Copy the full SHA
    18aad76 View commit details
    Browse the repository at this point in the history