3x-ui/docs/Tasktracking/2026-04-27-fix-batch-edit-shouldbindjson.md

# Task Record

Date: 2026-04-27
Related Module: web/controller/inbound.go
Change Type: Fix

## Background
批量编辑（batch edit）功能提示 "入站连接已成功更新 (invalid character 'i' looking for beginning of value)"，操作实际未生效。

根因：前端全局 axios 配置将 POST Content-Type 设为 `application/x-www-form-urlencoded`，请求拦截器用 `Qs.stringify` 将数据转为 URL-encoded 格式。但 `batchUpdateInboundClients` 处理器使用了 `c.ShouldBindJSON`，它只接受 JSON 格式。form-encoded 数据（如 `inboundId=123&...`）以 `i` 开头，导致 Go JSON 解析器报 `invalid character 'i' looking for beginning of value`。

项目中其他所有 handler（`addInbound`、`updateInbound`、`addClient` 等）均使用 `c.ShouldBind`（根据 Content-Type 自动选择绑定方式），仅在 batchUpdateInboundClients 中使用了 `ShouldBindJSON`，导致不一致。

## Changes
- `web/controller/inbound.go:481`: `c.ShouldBindJSON(&request)` → `c.ShouldBind(&request)`
- `web/controller/inbound.go:477-479`: 为匿名结构体字段添加 `form` 标签，确保 form 绑定能正确映射字段名
- `web/controller/inbound.go:482`: 绑定失败时使用 `invalidFormData` 替代 `inboundUpdateSuccess` 作为错误前缀，避免 "入站连接已成功更新 (错误)" 的语义矛盾
- `web/service/inbound.go:722,767`: `enable` 字段类型断言改为 comma-ok 模式，非 bool 值时返回明确错误而非 panic

## Impact
- `ShouldBind` 会根据 Content-Type 自动选择 form 绑定（浏览器请求）或 JSON 绑定（其他客户端），与项目其他 handler 一致
- 向后兼容：JSON 请求依然能被正确解析（`json` 标签仍保留）
- 绑定失败的错误提示更准确（"数据格式错误" 而非 "入站连接已成功更新"）
- 恶意构造的请求不再能通过类型断言 panic 导致进程崩溃
- 不影响数据库、API、配置

## Verification
- `gofmt -l -w .` — 无格式问题
- `go vet ./web/controller/... ./web/service/...` — 通过

## Risks And Follow-Up
- 低风险，与项目现有代码风格一致