一、封禁链路整条没接上(P0,已实测复现)
route.New 的 autoban.New 漏传 HoneypotEnabled/Threshold/Duration,而
RecordHoneypot 首行就是 `if !b.honeypotEnabled { return false }`
→ 蜜罐命中从不封禁、落库 banned 恒 false、IsBanned 永不为真。
同一批改动又把中间件里蜜罐路径的常规计数也跳过了,净效果是蜜罐路径
从「10 次封禁」退化为零计数零封禁 —— 是 regression 不是修复。
处理:抽 newBanner() 作为唯一装配入口并补齐参数;新增 banner_config_test
四个用例(全部经由 newBanner,不再自建 Banner)。变异验证:分别把
HoneypotEnabled 置 false、HoneypotThreshold 置 0,对应用例按预期失败。
二、诱捕语义(审计时漏掉,比上面更影响功能价值)
原实现对**所有方法**计数且阈值 1 → 扫描器 GET 一次假登录页就吃 403,
永远走不到「POST 提交账号密码」那一步,蜜罐的核心价值(拿到字典)归零。
处理:提交凭据 → RecordHoneypot(不受 enable_auto_ban 约束,立即封);
纯探测 → 常规 Record(阈值 10)。新增 TestHoneypotProbeDoesNotBanImmediately,
变异验证:把 GET 也塞进 RecordHoneypot,测试按预期失败。
三、落库边界(实测取证,非推断)
- Path 原样落库而列宽 512 → PG `SQLSTATE 22001 值太长了(512)`,整条证据丢失;
实测 600 字符失败、500 字符成功;
- 三处按字节截断 → `SQLSTATE 22021 无效的 "UTF8" 编码字节顺序`(中文密码/URL
可稳定触发)。统一改 rune 截断并按列宽逐项对齐。
四、资源与噪音
- alerted 无界 map → 带 TTL 的 deduper(上限 2 万,过期清理 + 超限硬裁),
同时承担告警 24h 冷却与纯探测落库采样(每 IP 每分钟一条)——
GET 洪泛不再变成等量 INSERT;
- credentialKeys 去掉 id/name,普通 JSON 探测不再被误判为凭据提交并触发假告警;
- canary 从 meta generator 移到 HTML 注释,generator 伪装成 WordPress,
不再同时向扫描器自曝「这是蜜罐」和「这站是 TamaBox」;
- serveLoginFailure 复用 WriteLoginPage,不再手抄响应头(已开始漂移);
- Describer 死代码 → Prefixes(),接到新命令 honeypot paths;
- honeypot_paths 解析挪到 honeypot.SplitPaths(CLI 与 route 共用),
补前导斜杠、拒绝 "/"(会让所有 404 变假登录页)。
五、凭据外置:数据库不再存用户名/密码
明文账号密码一旦入库就会跟着进备份、主从复制与任何能连库的报表工具,
事后无法彻底清除。改为:
- 新增 honeypot.CredRecord/CredStore:按天文件 creds-YYYY-MM-DD.jsonl,
目录 0700、文件 0600、坏行跳过、PurgeBefore 只删整天过期的文件;
- db.HoneypotHit 删掉 Username/Password/Fields/RawBody,只留 HadFields 元数据;
- HoneypotStore 删掉 TopCredentials/ListSinceByIP(CLI 改读文件后零调用);
- 新增 LegacyCredentialColumns/PurgeLegacyCredentialColumns:AutoMigrate 不删
既有列,老库升级后历史明文仍在 → 启动 WARN + `honeypot purge-creds` 显式 DROP;
- 清理器 HoneypotRetention 增加 CredPurgeable 参数,与 DB 记录共用保留期;
honeypot prune 也一并删过期凭据文件;
- CLI:hits 不再连库(DB 挂了也能查),stats 的凭据排行读文件;
- 配置 honeypot_cred_dir(缺省 data/honeypot-creds),.gitignore 加 /data/。
验证:go build ./... / go vet ./internal/... / go test ./...(含 PostgreSQL)全绿;
新增 credstore_test(追加/倒序/IP 过滤/坏行/Top/Purge/权限)、
TestHoneypotLegacyCredentialColumns(ADD COLUMN 模拟老库 → 检测 → DROP)、
TestHoneypotWritesCredentialsToFile;CLI 端到端手动验证 hits/--ip/--show-password。
162 lines
5.2 KiB
Go
162 lines
5.2 KiB
Go
// Copyright 2026 tamakyi. All rights reserved.
|
||
// Use of this source code is governed by a MIT-style
|
||
// license that can be found in the LICENSE file.
|
||
|
||
package db
|
||
|
||
import (
|
||
"context"
|
||
"testing"
|
||
"time"
|
||
|
||
"github.com/stretchr/testify/require"
|
||
)
|
||
|
||
// TestHoneypotStore 覆盖蜜罐命中记录的写入、检索与聚合。
|
||
//
|
||
// 重点验证「只追加」的审计语义:DeleteBefore 只按时间删,不做软删除;
|
||
// 以及 TopCredentials 能把攻击者正在用的字典聚合出来。
|
||
func TestHoneypotStore(t *testing.T) {
|
||
database, cleanup := newTestDB(t)
|
||
ctx := context.Background()
|
||
|
||
t.Cleanup(func() {
|
||
if err := cleanup("honeypot_hits"); err != nil {
|
||
t.Fatal(err)
|
||
}
|
||
})
|
||
|
||
store := NewHoneypotStore(database)
|
||
now := time.Now()
|
||
|
||
rows := []*HoneypotHit{
|
||
{IP: "1.2.3.4", Method: "GET", Path: "/wp-admin", Provider: "Azure", CreatedAt: now.Add(-3 * time.Hour)},
|
||
{IP: "1.2.3.4", Method: "POST", Path: "/wp-login.php", HadFields: true, CreatedAt: now.Add(-2 * time.Hour)},
|
||
{IP: "1.2.3.4", Method: "POST", Path: "/wp-login.php", HadFields: true, CreatedAt: now.Add(-time.Hour)},
|
||
{IP: "5.6.7.8", Method: "POST", Path: "/.ssh/id_rsa", HadFields: true, CreatedAt: now.Add(-30 * time.Minute)},
|
||
}
|
||
for _, r := range rows {
|
||
require.NoError(t, store.Create(ctx, r))
|
||
}
|
||
|
||
since := now.Add(-4 * time.Hour)
|
||
|
||
total, err := store.CountSince(ctx, since)
|
||
require.NoError(t, err)
|
||
require.Equal(t, int64(4), total)
|
||
|
||
list, err := store.ListSince(ctx, since, 10)
|
||
require.NoError(t, err)
|
||
require.Len(t, list, 4)
|
||
// 倒序返回
|
||
require.Equal(t, "5.6.7.8", list[0].IP)
|
||
|
||
sources, err := store.TopSources(ctx, since, 10)
|
||
require.NoError(t, err)
|
||
require.Len(t, sources, 2)
|
||
require.Equal(t, "1.2.3.4", sources[0].IP)
|
||
require.Equal(t, int64(3), sources[0].Count)
|
||
require.Equal(t, "Azure", sources[0].Provider)
|
||
|
||
paths, err := store.TopPaths(ctx, since, 10)
|
||
require.NoError(t, err)
|
||
require.Len(t, paths, 3)
|
||
require.Equal(t, "/wp-login.php", paths[0].Path)
|
||
require.Equal(t, int64(2), paths[0].Count)
|
||
|
||
// 保留期清理
|
||
deleted, err := store.DeleteBefore(ctx, now.Add(-90*time.Minute))
|
||
require.NoError(t, err)
|
||
require.Equal(t, int64(2), deleted) // 3 小时前、2 小时前两条被清
|
||
|
||
total, err = store.CountSince(ctx, since)
|
||
require.NoError(t, err)
|
||
require.Equal(t, int64(2), total)
|
||
}
|
||
|
||
// TestHoneypotLegacyCredentialColumns 覆盖「旧版凭据列」的检测与清除。
|
||
//
|
||
// 背景:凭据已改为只落文件(见 honeypot.CredRecord),结构体也删掉了
|
||
// username/password/fields/raw_body 四个字段。但 GORM 的 AutoMigrate
|
||
// 不会删除既有列,老库升级上来后历史明文仍然留在表里。
|
||
// 这个测试用 ADD COLUMN 模拟老库,验证检测与 DROP COLUMN 都真的生效。
|
||
func TestHoneypotLegacyCredentialColumns(t *testing.T) {
|
||
database, cleanup := newTestDB(t)
|
||
ctx := context.Background()
|
||
|
||
t.Cleanup(func() {
|
||
if err := cleanup("honeypot_hits"); err != nil {
|
||
t.Fatal(err)
|
||
}
|
||
})
|
||
|
||
store := NewHoneypotStore(database)
|
||
|
||
cols, err := store.LegacyCredentialColumns(ctx)
|
||
require.NoError(t, err)
|
||
require.Empty(t, cols, "新建的表不应有旧版凭据列")
|
||
|
||
// 模拟旧版本留下的列。
|
||
require.NoError(t, database.Exec("ALTER TABLE honeypot_hits ADD COLUMN username varchar(256)").Error)
|
||
require.NoError(t, database.Exec("ALTER TABLE honeypot_hits ADD COLUMN password varchar(256)").Error)
|
||
|
||
cols, err = store.LegacyCredentialColumns(ctx)
|
||
require.NoError(t, err)
|
||
require.ElementsMatch(t, []string{"username", "password"}, cols)
|
||
|
||
dropped, err := store.PurgeLegacyCredentialColumns(ctx)
|
||
require.NoError(t, err)
|
||
require.ElementsMatch(t, []string{"username", "password"}, dropped)
|
||
|
||
cols, err = store.LegacyCredentialColumns(ctx)
|
||
require.NoError(t, err)
|
||
require.Empty(t, cols, "清除后不应再检测到旧版凭据列")
|
||
}
|
||
|
||
// TestHoneypotSize 覆盖体积报告:行数、时间跨度、磁盘占用。
|
||
//
|
||
// 目的:让「蜜罐表有没有把数据库撑臃肿」成为运维随时能自己查的事。
|
||
func TestHoneypotSize(t *testing.T) {
|
||
database, cleanup := newTestDB(t)
|
||
ctx := context.Background()
|
||
|
||
t.Cleanup(func() {
|
||
if err := cleanup("honeypot_hits"); err != nil {
|
||
t.Fatal(err)
|
||
}
|
||
})
|
||
|
||
store := NewHoneypotStore(database)
|
||
now := time.Now()
|
||
|
||
for i := 0; i < 3; i++ {
|
||
require.NoError(t, store.Create(ctx, &HoneypotHit{
|
||
IP: "9.9.9.9",
|
||
Method: "POST",
|
||
Path: "/wp-login.php",
|
||
HadFields: true,
|
||
CreatedAt: now.Add(-time.Duration(i) * time.Hour),
|
||
}))
|
||
}
|
||
|
||
info, err := store.Size(ctx)
|
||
require.NoError(t, err)
|
||
require.Equal(t, int64(3), info.Rows)
|
||
require.NotNil(t, info.Oldest)
|
||
require.NotNil(t, info.Newest)
|
||
// 最早的一条是 2 小时前,最新的一条是刚刚。
|
||
require.True(t, info.Newest.After(*info.Oldest))
|
||
// 磁盘占用在 PostgreSQL 下应当可查且不为零(表刚建,至少有元数据开销)。
|
||
require.Greater(t, info.Bytes, int64(0))
|
||
|
||
// 清理后行数应随之下降,但磁盘占用不保证立刻变小
|
||
// (PostgreSQL 的 DELETE 只标记死元组,需 VACUUM 才回收)。
|
||
deleted, err := store.DeleteBefore(ctx, now.Add(-90*time.Minute))
|
||
require.NoError(t, err)
|
||
require.Equal(t, int64(1), deleted)
|
||
|
||
info2, err := store.Size(ctx)
|
||
require.NoError(t, err)
|
||
require.Equal(t, int64(2), info2.Rows)
|
||
}
|