Files
Havline/internal/logstore/rotate_test.go
T
mrleeandDevin 9d75fb1d3e
CI / verify (push) Canceled after 0s
fix: 收敛正确性缺口并补齐质量门禁
备份、隧道生命周期、日志、会话与设置项存在一批「静默失效」问题,本次一并收敛。

正确性:
- 备份归档补上 session_secret 与 frp 配置。缺密钥时换机恢复后库里所有凭据都解不开,
  界面表现为「配置都在、一执行就报解密失败」;同时支持口令加密(scrypt + AES-256-GCM)。
- 修复「立即备份」必失败:tar 头用的是遍历时的文件尺寸,而 logs、frpc 配置在打包期间
  还在增长,读出的字节多于声明值导致 archive/tar 报错、整份备份作废。改为打开后重新
  取尺寸并用 LimitReader 兜住,被截短时补零保持条目结构完整。
- 数据库改用 VACUUM INTO 生成一致性快照。直接打包正在写入的 SQLite 文件可能得到
  结构完整但内容自相矛盾的副本,恢复后才暴露。
- Cloudflare 启动改用 StartAuto,Shutdown 显式 StopAll,子进程绑定应用 ctx。此前
  重启会把用户手动停掉的隧道重新拉起,退出也可能留下孤儿进程。
- Nginx 访问/错误日志加轮转(200MB 阈值、gzip 归档、USR1 重开),读取改为从文件尾
  分块倒读,并跨当日归档统计。此前日志只增不减,读接口每次全量扫描。
- 过期会话改由日任务清理;删除反代规则时同步清掉流量累计,避免同名规则重建继承旧读数。
- 启动加就绪门:Nginx bootstrap 完成后再放行 FRP 对账、Agent 隧道与 Cloudflare 自启。

设置项:
- 新增设置项注册表(internal/settings/registry.go),白名单与默认值从它派生,
  并记录每个键的消费者,防止再出现「界面能改、后端没人读」的黑洞设置。
- 接上三个此前不生效的设置项:timezone(启动与保存时应用到 time.Local)、
  theme(后端为权威源,挂载时覆盖本地偏好、顶栏切换回写)、log_retention_days
  (控制日志归档保留天数)。

工程质量:
- gofmt 全量格式化 23 个历史文件,CI 增加格式化门禁。
- 前端引入 ESLint(flat config)与 lint 脚本,CI 增加 lint 步骤;顺带修掉
  logDisplay 的 let→const 与 LogsView 模板内直接改 prop 两处问题。
- 删除无引用的 pnpm-lock.yaml,prototype/ 加入忽略列表。
- 打包版本对齐:Docker manifest 同步到 2.0.0,CI 校验 VERSION 与 package.json 一致。

验证:gofmt 0 文件、go vet、go test ./...、npm run lint/typecheck/test/build 全绿。

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
2026-10-24 16:30:00 +08:00

189 lines
5.2 KiB
Go
Raw Blame History

This file contains ambiguous Unicode characters
This file contains Unicode characters that might be confused with other characters. If you think that this is intentional, you can safely ignore this warning. Use the Escape button to reveal them.
package logstore
import (
"compress/gzip"
"io"
"os"
"path/filepath"
"strings"
"testing"
"time"
)
// TestRotateIfNeededSkipsSmallFile:未超阈值不能产生任何副作用,否则每小时一次的
// 定时任务会把日志切成无数小文件。
func TestRotateIfNeededSkipsSmallFile(t *testing.T) {
dir := t.TempDir()
path := filepath.Join(dir, "access.log")
if err := os.WriteFile(path, []byte("small\n"), 0o640); err != nil {
t.Fatal(err)
}
reopened := 0
r := NewRotator(path, 0, func() error {
reopened++
return nil
})
rotated, err := r.RotateIfNeeded(time.Now())
if err != nil {
t.Fatal(err)
}
if rotated {
t.Fatal("未超阈值不应轮转")
}
if reopened != 0 {
t.Fatal("未轮转时不应通知重开日志")
}
if _, err := os.Stat(path); err != nil {
t.Fatalf("原文件应保持不动:%v", err)
}
}
// TestRotateCompressesAndReopens:超阈值时要归档成 .gz、通知进程重开文件、并删掉原文件。
func TestRotateCompressesAndReopens(t *testing.T) {
dir := t.TempDir()
path := filepath.Join(dir, "access.log")
content := strings.Repeat("2026-10-24T10:00:00+08:00 a.example.com 18080 GET / 200 0.010 1.2.3.4 10.0.0.2:80 100 200\n", 200)
if err := os.WriteFile(path, []byte(content), 0o640); err != nil {
t.Fatal(err)
}
reopened := 0
r := NewRotator(path, 0, func() error {
reopened++
return nil
})
r.MaxBytes = 1024 // 强制触发
now := time.Date(2026, 10, 24, 13, 5, 0, 0, time.Local)
rotated, err := r.RotateIfNeeded(now)
if err != nil {
t.Fatal(err)
}
if !rotated {
t.Fatal("超过阈值应轮转")
}
if reopened != 1 {
t.Fatalf("轮转后必须通知重开日志,实际调用 %d 次", reopened)
}
if _, err := os.Stat(path); !os.IsNotExist(err) {
t.Fatal("轮转后原文件应被移走")
}
archive := filepath.Join(dir, "access-20261024-1305.log.gz")
file, err := os.Open(archive)
if err != nil {
t.Fatalf("应生成归档文件 %s:%v", archive, err)
}
defer file.Close()
gz, err := gzip.NewReader(file)
if err != nil {
t.Fatal(err)
}
defer gz.Close()
data, err := io.ReadAll(gz)
if err != nil {
t.Fatal(err)
}
if string(data) != content {
t.Fatalf("归档内容应与原文件一致:got %d bytes want %d", len(data), len(content))
}
}
// TestRotatePrunesOldArchives:归档份数与保留天数都要生效。
func TestRotatePrunesOldArchives(t *testing.T) {
dir := t.TempDir()
path := filepath.Join(dir, "access.log")
// 造 3 个既有归档,其中 2 个已经超期
old := time.Now().Add(-40 * 24 * time.Hour)
stale := []string{"access-20200101-0000.log.gz", "access-20200102-0000.log.gz"}
for _, name := range stale {
p := filepath.Join(dir, name)
if err := os.WriteFile(p, []byte("x"), 0o640); err != nil {
t.Fatal(err)
}
if err := os.Chtimes(p, old, old); err != nil {
t.Fatal(err)
}
}
fresh := filepath.Join(dir, "access-20261024-1200.log.gz")
if err := os.WriteFile(fresh, []byte("x"), 0o640); err != nil {
t.Fatal(err)
}
if err := os.WriteFile(path, []byte(strings.Repeat("line\n", 500)), 0o640); err != nil {
t.Fatal(err)
}
r := NewRotator(path, 7, nil) // 保留 7 天
r.MaxBytes = 1024
if _, err := r.RotateIfNeeded(time.Now()); err != nil {
t.Fatal(err)
}
for _, name := range stale {
if _, err := os.Stat(filepath.Join(dir, name)); !os.IsNotExist(err) {
t.Fatalf("超过保留天数的归档应被删除:%s", name)
}
}
if _, err := os.Stat(fresh); err != nil {
t.Fatalf("未超期的归档应保留:%v", err)
}
}
// TestTailLinesReadsFromEnd:尾部读取既要拿对最后几行,也不能因为文件很大而读全量。
func TestTailLinesReadsFromEnd(t *testing.T) {
dir := t.TempDir()
path := filepath.Join(dir, "access.log")
var b strings.Builder
for i := 0; i < 5000; i++ {
b.WriteString("line-")
b.WriteString(strings.Repeat("x", 20))
b.WriteString("\n")
}
b.WriteString("last-line-a\nlast-line-b\n")
if err := os.WriteFile(path, []byte(b.String()), 0o640); err != nil {
t.Fatal(err)
}
lines, err := tailLines(path, 3)
if err != nil {
t.Fatal(err)
}
want := []string{strings.TrimSpace("line-xxxxxxxxxxxxxxxxxxxx"), "last-line-a", "last-line-b"}
if len(lines) != len(want) {
t.Fatalf("应返回 3 行,实际 %d:%v", len(lines), lines)
}
for i := range want {
if lines[i] != want[i] {
t.Fatalf("第 %d 行不符:got %q want %q", i, lines[i], want[i])
}
}
}
// TestTailLinesSingleLongLine:单行超过初始块大小时不能被截断成多行。
func TestTailLinesSingleLongLine(t *testing.T) {
dir := t.TempDir()
path := filepath.Join(dir, "access.log")
long := strings.Repeat("y", 300*1024)
if err := os.WriteFile(path, []byte(long+"\n"), 0o640); err != nil {
t.Fatal(err)
}
lines, err := tailLines(path, 5)
if err != nil {
t.Fatal(err)
}
if len(lines) != 1 || lines[0] != long {
t.Fatalf("超长单行应完整返回,实际 %d 行(首行长度 %d)", len(lines), len(lines[0]))
}
}
// TestTailLinesMissingFile:日志还没产生时返回空而不是报错。
func TestTailLinesMissingFile(t *testing.T) {
lines, err := tailLines(filepath.Join(t.TempDir(), "nope.log"), 10)
if err != nil {
t.Fatalf("文件不存在不应报错:%v", err)
}
if len(lines) != 0 {
t.Fatalf("应返回空行列表,实际 %v", lines)
}
}