mpyw/txbypass

TXBYPASS

去找个报告它在交易中用过数据库句柄的linter。

概述

回调以 tx 的形式接收该事务。r.db 是交易来自的池，因此审计日志写入事务外部。

代码：出了什么问题

写入能经回滚存活。它从未包含在交易中

写入可以无限等待。它运行在另一个连接上，需要该事务持有的锁。对于一个连接池，它会等待连接本身

读取无法看到事务的行。它们尚未提交

代码会编译。一个没有失败的测试也通过了。txbypass 从源代码中找到了它。

每条规则的报告都以规则名称结尾。这个名称就是 //txbypass:ignore 所采用的。关于指令本身的报告不命名规则，且无忽略会使其静音。

安装

Go linter that reports a database handle used inside a transaction it opened.

Overview

package app

import (
	"context"

	"gorm.io/gorm"
)

func (r *OrderRepo) Place(
	ctx context.Context,
	o *Order,
) error {
	return r.db.WithContext(ctx).Transaction(
		func(tx *gorm.DB) error {
			if err := tx.Create(o).Error; err != nil {
				return err
			}
			log := &AuditLog{OrderID: o.ID}
			return r.db.Create(log).Error
		},
	)
}

$ txbypass ./...
order.go:19:11: r.db bypasses the transaction opened with Transaction; use tx instead [outer-handle]

The callback receives the transaction as tx. r.db is the pool the transaction came from, so the audit log is written outside the transaction.

The code compiles. A test where nothing fails passes too. txbypass finds it from the source.

Every report of a rule ends with the rule’s name. That name is what //txbypass:ignore takes. A report about a directive itself names no rule, and no ignore silences it.

Install

txbypass ./...