tamakyi
b4c2a8452f
audit-2026-09-21.md 里的后端条目落地。每条都配了回归守卫，并逐条做过变异验证 （注入原 bug → 确认按正确理由变红 → 还原 → 变绿）。 安全 - CSRF 只加同源 Origin/Referer 守卫（csrf.go），不动模板 - 会话 cookie 的 Secure 按请求判断；两步验证两闸（loginGuard + 验证码一次性） - 跳转收口到 safeInternalPath；clientIP 只认 XFF 最后一段 - GPX 按属主圈定且查 RowsAffected；下载名走 downloadDisposition - 上传口统一 MaxBytesReader；落盘后的失败路径清掉半成品 数据一致性 - 区域判定两端各过一遍再 AND（apiTrack 与 movementsWhere 对齐） - Filter.Verdict 全域一致（下推到 visitsWhere）；聚合刷新必须返回 error - 附属数据改挂坐标键，Go 与 SQL 两套表达式逐位一致 - 脏备份的伪时间/越界坐标不再进领域模型；rows 循环统一 drainRows - 合并与导出：writeRawRow 上抛错误、悬空引用清理、原生列补全 性能 - placeCols 相关子查询带 dataset_id 前导列；findSourceArchive 先按大小筛 - 批量勾选改 ANY($3::bigint[])；fillupCache 加上限；同一查询不再查两遍 另：新增「全新库」冒烟测试 —— 空库跑完 Migrate 再用应用侧读查询真打一遍， 迁移块排到依赖表之前这种错只有新库会炸，老库毫无症状。
24 lines
797 B
Go
24 lines
797 B
Go
package web
|
||
|
||
import (
|
||
"os"
|
||
"strings"
|
||
"testing"
|
||
)
|
||
|
||
// 统计页的 Weekday 只该查一次。
|
||
//
|
||
// 原实现是「在 runJobs 里注册了 weekday 任务」之外，又在渲染前同步查了一遍
|
||
// （`d.Weekday, err = s.q.Weekday(ctx, f)`），等于同一个查询发了两次——纯浪费。
|
||
// 这条守卫盯住「不许再出现第二次」。
|
||
func TestStatsHandlerQueriesWeekdayOnce(t *testing.T) {
|
||
b, err := os.ReadFile(pkgFile(t, "handlers_public.go"))
|
||
if err != nil {
|
||
t.Fatalf("读取 handlers_public.go 失败: %v", err)
|
||
}
|
||
if n := strings.Count(string(b), "s.q.Weekday("); n != 1 {
|
||
t.Errorf("handlers_public.go 里 s.q.Weekday( 应恰好出现 1 次，实际 %d 次"+
|
||
"（>1 说明又重复查了；0 说明写法改了、这条守卫需要同步更新）", n)
|
||
}
|
||
}
|