sql审计必须劫持sql.driver或注册代理驱动,而非仅封装函数;需标准化查询、解析ast识别低效模式;gorm场景应实现gorm.logger接口;优化建议须含调用栈、服务名、耗时、行数等上下文。

SQL审计必须拦截database/sql的底层调用,不能只包一层函数
Go 的 database/sql 是接口抽象层,真正执行 SQL 的是驱动(如 mysql、postgres)。如果只在业务层封装一个 ExecWithAudit() 函数,会漏掉 ORM(如 GORM)、raw db.Query()、事务内嵌套调用等场景。审计要生效,必须替换或劫持 sql.Driver 或使用 sql.Register() 注册代理驱动。
实操建议:
- 用
sql.Register("audit-mysql", &auditDriver{driver: mysql.MySQLDriver{}})包装原驱动,所有sql.Open("audit-mysql", ...)请求都会经过审计逻辑 - 在
auditDriver.Open()返回的*sql.Conn上,重写PrepareContext()和ExecContext(),提取query字符串和args - 避免在审计逻辑中做耗时操作(如写磁盘、发 HTTP),应异步投递到 channel 或 ring buffer,否则拖慢主流程
识别低效 SQL 不能只靠关键词匹配,得解析 AST 或至少做模式归一化
简单用 strings.Contains(query, "SELECT *") 或正则匹配 "ORDER BY.*LIMIT [0-9]+,[0-9]+" 容易误报或漏报。比如 SELECT * FROM users WHERE id = ? 没问题,但 SELECT * FROM logs WHERE created_at > ? 就危险;又比如 LIMIT 10 OFFSET 10000 是典型深分页,但 LIMIT 10 单独出现未必有问题。
实操建议:
- 对 query 做标准化:去除换行/多余空格、统一大小写、替换字面量为
?(如"WHERE id = 123"→"WHERE id = ?"),再匹配模板规则 - 用轻量 parser 如
github.com/xwb1989/sqlparser提取SELECT字段列表、WHERE条件、ORDER BY、LIMIT等结构,判断是否含非索引字段、无WHERE、ORDER BY字段未建索引等 - 记录执行耗时 + 扫描行数(需驱动支持
RowsAffected()或从 MySQLSHOW PROFILES补充),超阈值(如 >100ms 或扫描 >1000 行)才触发优化建议
GORM 场景下审计需额外处理预编译语句和钩子注入
GORM v2 默认启用 PrepareStmt: true,所有查询都走 Prepare/Query/Close 流程,且内部大量使用 context.WithValue() 透传 session 信息。若只监听 Exec,会错过 First()、Find() 等方法生成的隐式查询;同时 GORM 的 AfterFind、BeforeUpdate 钩子也可能触发额外 SQL,不纳入审计就断链。
Go 配置库,使用 spf13/viper — 分层优先级(flag > env >file > KV > default),提供 BindPFlag/BindPFlags、SetEnvPrefix + SetEnvKeyReplace 等功能。
实操建议:
- 启用 GORM 的
logger接口(实现gorm.Logger),在Trace()方法里拿到归一化后的sql和rows,比劫持驱动更稳定 - 用
db.Session(&gorm.Session{Context: ctx})在关键路径注入自定义 context key,让钩子函数能回调审计模块记录来源 - 注意 GORM 的
Count()查询常被忽略——它可能触发全表扫描,但不在主业务 SQL 审计流里,需单独加规则检测
优化建议必须带上下文,否则开发无法判断是否真该改
只返回“建议添加索引”或“避免 SELECT *”没用。开发者需要知道:这个 SQL 在哪个 service、哪个 handler、调用频次多少、平均延迟多少、最近 1 小时是否突增、对应表当前行数、主键和现有索引有哪些。
实操建议:
- 审计日志结构至少包含:
sql_normalized、stack(调用栈前 3 层)、service_name、http_path(若在 HTTP handler 中)、duration_ms、rows_scanned、table_names(从 AST 解析出) - 对“缺失索引”类建议,自动查
information_schema.STATISTICS,输出类似:"users.created_at 无索引,该表当前 2.4M 行,近 5 分钟此 SQL 调用 142 次,平均 327ms" - 避免建议冲突:同一张表上已有
(a,b)复合索引,就别再建议单独给a加索引;用SHOW INDEX结果做覆盖判断
真正难的不是抓 SQL,而是把一条慢查映射回具体代码位置、判断影响范围、排除误报。没有调用栈和指标上下文的审计,最后只会变成没人看的日志文件。
golang免费学习笔记(深入):立即使用
在学习笔记中,你将探索golang的核心概念和高级技巧!










