监测->告警->WebHook添加失败

huixia0010

看前端请求是这样的：
curl 'https://vx.link/openapi/v1/vxping' -X POST --data-raw 'token=xxxxx&action=trigger_add&type=equal&val=0&check_type=avg&ping_id=7479&trigger_type=webhook&trigger_params=https%3A%2F%2Fxxxx.com%2Falert.php'
后端直接 524了。

隐私部分xxxx处理了，看起来这个添加按钮的响应有Bug，并不能按预期工作。

响应的PHP代码：

<?php
$json = file_get_contents('php://input');
$data = json_decode($json, true);
if (isset($data['action']) && $data['action'] === 'ping') {
    http_response_code(200);
    echo 'OK';
    exit;
}

huixia0010

echo 'OK'的部分我改成输出200，服务端报错是：

{
    "data": "WebHook 地址没有正确返回 200",
    "status": 0
}

我很迷惑，没返回200，是否能附带些服务器报错信息啥的方便用户定位自己的问题？

huixia0010

建议UI添加告警的文字描述调整下：

响应 Ping 请求
1.在添加 WebHook 时，这个地址必须响应一次application/x-www-form-urlencoded action=ping 的 POST 请求，其HTTP状态返回值为 200，内容为200。

2.监测点的名字支持修改

3.UI修正Bug：添加成功一个告警后，无法添加其他项目的告警，刷新后才能添加。

4.告警并不是向webhook发的json，而是发的表单，请修改会向 WebHook 地址发送一个 JSON 格式的数据

5.status发的是中文不是什么happen、wait

微林

huixia0010 感谢反馈，关于此问题，我们会在近期更新文档，给出更详细的说明。

微林

huixia0010 目前已经根据您反馈的情况做了一部的调整，详情可以在这里查阅。
关于您提到的只能添加一个项目的告警的 bug，目前暂时无法复现出来，如果可以的话，请您录制一个屏幕录像，然后上传到 tmp.link 再贴到帖子里，以便我们了解详细的情况。

huixia0010

微林你这样复现，新增6个监控节点，然后给第一个加丢包率100，webhook到某个url。
加完给第二个加，此时，默认只有丢包率具体数值需要重新填写。填了点右下角对勾添加。
再给第三个加，我这边复现是没办法完整的连续的给6个节点都加好监控，加到第三个就一直转圈圈了。

此时，刷新页面，页面会显示正在处理，而无法完整显示全部数据。

微林

huixia0010
此问题目前已被修复，原因是因为当发生错误时，调用了错误的处理函数。
此外，关于直接修改监测点名字的需求也已完成。

所有更新日志可以在这里查阅。

huixia0010

微林修改标题这个有点粗糙：
1.点了编辑按钮直接给了个空输入框。不人性化，建议编辑框保留旧内容。
2.如果留空，回车，会直接置空。需要做校验。
3.响应ESC取消编辑

我提这么多建议，有没有热心用户奖励，略略略~

微林

huixia0010 已收到您的建议，我们会在近期做出调整。