先把「实机复测」能在这台 Linux 机器上验的全部自己验掉(真服务端 + 真 Chromium + 真样本),
六批全部通过,另挖出 16 条缺陷,逐条修完并复验:
导入
- 坏字节被静默替换成 U+FFFD 却报「组别不存在」→ 检测 RuneError 回 400 + 可操作中文文案
- UTF-16 无 BOM 误报「缺少列 event_code」→ 保守判据识别并正确解码
- CSV 语法错只有英文 → 中文提示,英文原文保留在 detail
- 表头缺列补「第 1 行应是表头」的指向
导出
- 名次列宽 8 → 11(放得下「3(并列)」,Excel 里不再硬裁)
- 运动员/队列宽 18 → 24(含奖牌明细的同一列)
- 聚合流水行补组别(仅单位在该小项只参加一个组别时回推;多组别宁可留空也不猜)
- docProps 作者从 excelize 默认的 xuri 改成中性名 + 真实导出时间
- Content-Disposition 补 ASCII 退化 filename(9 个下载端点,含此前没有测试的 project.zip)
错误分级与限流
- 未知大屏指令 500 → 400 VALIDATION_FAILED(传输层真失败仍走 5xx)
- 5xx 的 ERROR 日志补 role / ip(4xx 的 WARN 一直有)
- 写限流 30 → 120 次/分钟/用户(登录 5 次/分/IP 不动,性质不同)
- 赛事名含非法字符时导出给 400 + 可操作提示(原来永远 500「服务端异常」)
大屏与基础设施
- 实时赛况 2 栏成绩末位被裁(1:09.00 显示成 1:09.0)→ 成绩列按内容定宽 + nowrap;
字号不缩,让位给可省略号截断的运动员列。真 Chromium 实测 slack −45px → +6px、0 裁切、
scrollWidth 恒 1920(无横向滚动条);顺带修掉 1 栏长成绩同样被裁 8px 的问题
- logging 启动即预建两个日志目录(原来 interceptions/ 懒创建,现场会以为日志坏了)
- C→S 裸帧由静默忽略改为 WARN,并把「要套 {event,data} 外壳」写进日志
- 建单位/大项漏传 active 不再静默建成「停用」(新增 decodeJSONPresence 按键是否存在判意图)
文档
- README 单位 logo 措辞收窄到「大屏端」(管理端只负责上传配置)
- 导入规范补三类编码报错说明 + 现场另存结论 + 限流提示
- 部署指南补「导出前先看赛事名称」与「切换后怎么确认真的切过去了」
- devdocs 05/08/13 同步(含第三十五轮的完整记录与教训)
自检:服务端 13 包连跑两遍全绿;前端 257 例;vue-tsc + build 通过;Console 18 文件。
版本号 dev_0.1.4 → dev_0.1.5。
167 lines
6.5 KiB
Go
167 lines
6.5 KiB
Go
package httpapi_test
|
||
|
||
import (
|
||
"bytes"
|
||
"errors"
|
||
"log/slog"
|
||
"net/http"
|
||
"os"
|
||
"path/filepath"
|
||
"strings"
|
||
"testing"
|
||
"time"
|
||
|
||
"git.yeij.top/AskaEth/Scoreboard/server/internal/httpapi"
|
||
"git.yeij.top/AskaEth/Scoreboard/server/internal/logging"
|
||
)
|
||
|
||
// fileLogEnv 把服务端日志按「放行 / 拦截」两类真的写进临时目录,返回两个目录路径。
|
||
// 到这里为止的用例都只看内存缓冲,这一份是「真的读文件」的验收:分流、级别、落盘都要落地。
|
||
func fileLogEnv(t *testing.T) (*testEnv, string, string) {
|
||
t.Helper()
|
||
dir := t.TempDir()
|
||
handler, closer := logging.New(logging.Options{
|
||
Dir: dir,
|
||
Level: slog.LevelInfo,
|
||
MaxSizeMB: 20,
|
||
KeepFiles: 10,
|
||
Console: &bytes.Buffer{},
|
||
})
|
||
t.Cleanup(func() { _ = closer.Close() })
|
||
env := newTestEnvWith(t, func(o *httpapi.Options) { o.Logger = slog.New(handler) })
|
||
return env, filepath.Join(dir, logging.DirOperations), filepath.Join(dir, logging.DirInterceptions)
|
||
}
|
||
|
||
// readCategory 读某类日志今天的文件;文件还没建出来时返回空串。
|
||
func readCategory(t *testing.T, dir, prefix string) string {
|
||
t.Helper()
|
||
path := filepath.Join(dir, prefix+"-"+time.Now().Format("20060102")+".log")
|
||
raw, err := os.ReadFile(path)
|
||
if os.IsNotExist(err) {
|
||
return ""
|
||
}
|
||
if err != nil {
|
||
t.Fatalf("读日志 %s: %v", path, err)
|
||
}
|
||
return string(raw)
|
||
}
|
||
|
||
// waitCategory 等到某类日志出现目标片段(审计 INFO 在响应写完之后才落)。
|
||
func waitCategory(t *testing.T, dir, prefix, want string) string {
|
||
t.Helper()
|
||
for i := 0; i < 200; i++ {
|
||
got := readCategory(t, dir, prefix)
|
||
if strings.Contains(got, want) {
|
||
return got
|
||
}
|
||
time.Sleep(5 * time.Millisecond)
|
||
}
|
||
t.Fatalf("等不到 %s/%s 里出现 %q,当前内容:\n%s", dir, prefix, want, readCategory(t, dir, prefix))
|
||
return ""
|
||
}
|
||
|
||
// 中间件层的拦截(403/401 这类发生在 handler 之前的拦截)必须落进 interceptions/,
|
||
// 带中文原因与指向中间件的拦截点;同时验证两类文件互不串门、5xx 只进拦截类。
|
||
func TestMiddlewareInterceptsLandInInterceptionFile(t *testing.T) {
|
||
env, opsDir, interDir := fileLogEnv(t)
|
||
env.seed(t)
|
||
|
||
// recorder 访问 admin-only 的只读接口 → RequireRole 403。
|
||
env.createUser("recorder", "recorder-pass", "recorder")
|
||
recorder, _ := env.client()
|
||
env.login(recorder, "recorder", "recorder-pass")
|
||
if status, _ := env.do(recorder, http.MethodGet, "/api/audit-logs", nil, false); status != http.StatusForbidden {
|
||
t.Fatalf("recorder 读审计应 403,得到 %d", status)
|
||
}
|
||
|
||
// 已登录但不带 CSRF 头的写请求 → CSRF 中间件 403。
|
||
if status, _ := env.do(recorder, http.MethodPost, "/api/teams",
|
||
map[string]any{"code": "X", "name": "越权单位"}, false); status != http.StatusForbidden {
|
||
t.Fatalf("缺 CSRF 的写请求应 403,得到 %d", status)
|
||
}
|
||
|
||
// 必须改密的账号写操作 → MustChangePassword 403。
|
||
lockedID := env.createUser("locked", "locked-pass", "recorder")
|
||
if _, err := env.sys.Exec(`UPDATE users SET must_change_password = 1 WHERE id = ?`, lockedID); err != nil {
|
||
t.Fatalf("设置 must_change_password: %v", err)
|
||
}
|
||
locked, _ := env.client()
|
||
env.login(locked, "locked", "locked-pass")
|
||
if status, _ := env.do(locked, http.MethodPost, "/api/teams",
|
||
map[string]any{"code": "Y", "name": "越权单位"}, true); status != http.StatusForbidden {
|
||
t.Fatalf("改密前写操作应 403,得到 %d", status)
|
||
}
|
||
|
||
// 未登录的写请求 → RequireWrite 401。
|
||
guest, _ := env.client()
|
||
if status, _ := env.do(guest, http.MethodPost, "/api/teams",
|
||
map[string]any{"code": "Z", "name": "未登录"}, false); status != http.StatusUnauthorized {
|
||
t.Fatalf("guest 写请求应 401,得到 %d", status)
|
||
}
|
||
|
||
inter := waitCategory(t, interDir, logging.DirInterceptions, "RequireRole")
|
||
for _, want := range []string{
|
||
"请求被拦截",
|
||
"无权访问该资源",
|
||
"请先登录",
|
||
"CSRF 校验失败,请刷新页面重试",
|
||
"请先修改初始密码",
|
||
"internal/middleware/middleware.go:",
|
||
} {
|
||
if !strings.Contains(inter, want) {
|
||
t.Fatalf("interceptions 应含 %q,得到:\n%s", want, inter)
|
||
}
|
||
}
|
||
// 四个中间件各留下一条 WARN,拦截点都在 middleware.go、都不是 response.go。
|
||
for _, fn := range []string{"RequireRole", "RequireWrite", "CSRF", "MustChangePassword"} {
|
||
if !strings.Contains(inter, fn) {
|
||
t.Fatalf("interceptions 应含拦截点函数 %s,得到:\n%s", fn, inter)
|
||
}
|
||
}
|
||
if strings.Contains(inter, "response.go") {
|
||
t.Fatalf("中间件拦截的拦截点不该落在 response.go,得到:\n%s", inter)
|
||
}
|
||
|
||
// 放行类里不能出现这些拦截行。
|
||
ops := readCategory(t, opsDir, logging.DirOperations)
|
||
for _, unwanted := range []string{"请求被拦截", "无权访问该资源", "CSRF 校验失败"} {
|
||
if strings.Contains(ops, unwanted) {
|
||
t.Fatalf("operations 不该含拦截日志 %q,得到:\n%s", unwanted, ops)
|
||
}
|
||
}
|
||
// 放行类里应当有登录这条 INFO(证明两类文件各自都有内容)。
|
||
if !strings.Contains(waitCategory(t, opsDir, logging.DirOperations, "操作已放行"), `"action":"auth.login"`) {
|
||
t.Fatalf("operations 应含放行日志,得到:\n%s", readCategory(t, opsDir, logging.DirOperations))
|
||
}
|
||
}
|
||
|
||
// 5xx 的归属:只进 interceptions/,不进 operations/。
|
||
func TestServerErrorGoesToInterceptionFileOnly(t *testing.T) {
|
||
dir := t.TempDir()
|
||
// 未知 action 现在是 4xx;这里用真正的下发失败(写 socket 失败)造 5xx。
|
||
sender := &strictDisplaySender{failWith: errors.New("写 socket 失败: broken pipe")}
|
||
handler, closer := logging.New(logging.Options{Dir: dir, Level: slog.LevelInfo, Console: &bytes.Buffer{}})
|
||
defer func() { _ = closer.Close() }()
|
||
env := newTestEnvWith(t, func(o *httpapi.Options) {
|
||
o.Logger = slog.New(handler)
|
||
o.Display = sender
|
||
})
|
||
env.createUser("recorder", "recorder-pass", "recorder")
|
||
client, _ := env.client()
|
||
env.login(client, "recorder", "recorder-pass")
|
||
|
||
if status, _ := env.do(client, http.MethodPost, "/api/display/command",
|
||
map[string]any{"action": "goto", "payload": map[string]any{"page": "medals"}}, true); status != http.StatusInternalServerError {
|
||
t.Fatalf("下发失败应 500,得到 %d", status)
|
||
}
|
||
|
||
inter := waitCategory(t, filepath.Join(dir, logging.DirInterceptions), logging.DirInterceptions, "请求失败")
|
||
if !strings.Contains(inter, "displayCommandHandler") {
|
||
t.Fatalf("ERROR 行应带拦截点,得到:\n%s", inter)
|
||
}
|
||
ops := readCategory(t, filepath.Join(dir, logging.DirOperations), logging.DirOperations)
|
||
if strings.Contains(ops, "请求失败") {
|
||
t.Fatalf("5xx 不该进 operations,得到:\n%s", ops)
|
||
}
|
||
}
|