elastic/kibana

[EQL] Remove usage of ignore:400 for syntax validation

开放

#169,042 创建于 2023年10月16日

在 GitHub 查看贡献者指南
 (5 条评论) (0 个反应) (1 位负责人)TypeScript (8,021 个派生)batch import
Team: SecuritySolutionTeam:Detection Engineeringbuggood first issue

仓库指标

星标
 (19,065 个星标)
PR 合并指标
 (PR 指标待抓取)

描述

Describe the bug:

Currently, the EQL search strategy adds "ignore": [400] to the params sent to the elasticsearch-js client which causes the client to treat 400 errors as expected:

https://github.com/elastic/kibana/blob/6efef0496077f4e61d49e4e43f75941ff3d98d9e/x-pack/plugins/security_solution/public/common/hooks/eql/api.ts#L42

As a result, the response back may indeed be a 400 error but it is returned as a normal 200 response.

This may have been necessary at some point but now ES properly sends a message back indicating syntax errors:

image

贡献者指南

研究方向
该问题需要修改文件 `x pack/plugins/security solution/public/common/hooks/eql/api.ts`，从请求选项中移除 `'ignore': [400]` 参数。移除后，确保 400 错误能正确作为错误而非成功响应被抛出。此改动范围小且集中。
技术栈
typescript
领域
backend
议题类型
缺陷
难度面向新贡献者的预计实现难度，1 表示很小改动，5 表示专家级工作。
2
预计时间有经验贡献者完成调查、实现、测试并准备 pull request 的粗略时间范围。
1-3 小时
活动状态议题当前的可参与程度：新鲜、活跃、陈旧、阻塞或等待维护者输入。
活跃
清晰度议题是否清楚说明期望改动、验收标准和下一步。
清晰
前置要求
JavaScriptElasticsearch
新手友好度1-100 的估计分数，表示该议题对首次贡献者的友好程度。
75