Files
panel/internal/data/database.go
T
耗子andClaude Opus 5 e0588a1468 refactor: 把 context 打通到数据库、对象存储与容器 API
上一轮打通 shell 路径时留下三处接口本身不带 ctx,导致底下的
context.Background() 无法消除。这三处 contextcheck 都不报(接口方法没有
ctx 形参,静态分析认为合理),属于 lint 发现不了的死角:

- pkg/db.Operator 的 17 个方法加 ctx,MySQL/Postgres/ClickHouse/SQLite/
  MongoDB/Elasticsearch 实现同步改。其中 Elasticsearch 的 resty 请求原先
  根本没挂 ctx,不只是传了 Background
- pkg/storage.Storage 的 6 个方法加 ctx,local/s3/sftp/webdav 四个实现与
  s3sdk 子包同步改;sftp 连接改用 net.Dialer.DialContext,连接阶段可取消
- internal/data/container*.go 的 moby client 调用,Container/Image/Network/
  Volume 四个 Repo 共 26 个方法加 ctx

非测试代码的 context.Background() 从 71 处降到 14 处,剩下的是应用根 ctx、
有意的独立超时,以及几个瞬时本机操作的叶子函数。双平台 lint 保持 0 issues。

新增 23 处 context.WithoutCancel,其中两处的理由值得记:

备份上传断开取消链,是因为 ClearStorageExpired 按 mtime 保留最新 N 份,
上传被取消留下的截断文件会被当成最新的有效备份,把真正完好的旧备份挤出
保留窗口——静默的数据丢失。而 Put 失败路径没有任何远端清理逻辑,本地侧
反而有(任务队列独享 TMPDIR + defer RemoveAll),两边不对称。

Postgres.UserDrop 是 REASSIGN OWNED → DROP OWNED → DROP USER 三步,中途
取消会留下丢了所有权却没被删掉的用户;MySQL 权限变更后的 FLUSH 被跳过则
新授权根本不生效。

ImagePull 经核实不需要断开:moby 的 LayerDownloadManager 把未完成的层写在
临时文件里并在取消时删除,层只有完整校验后才注册进 layer store,取消不会
留下悬空层。

顺带删掉 8 处 //nolint:contextcheck,它们当初就是因为 Operator 接口没有
ctx 才挂上的。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
2026-09-15 01:17:02 +08:00

120 lines
3.2 KiB
Go

package data
import (
"context"
"fmt"
"slices"
"gorm.io/gorm"
"github.com/acepanel/panel/v3/internal/biz"
"github.com/acepanel/panel/v3/pkg/db"
)
type databaseRepo struct {
db *gorm.DB
}
func NewDatabaseRepo(db *gorm.DB) biz.DatabaseRepo {
return &databaseRepo{
db: db,
}
}
// ListServers 列出数据库服务器
func (r *databaseRepo) ListServers(typ string) ([]*biz.DatabaseServer, error) {
var databaseServer []*biz.DatabaseServer
query := r.db.Model(&biz.DatabaseServer{}).Order("id desc")
if typ != "" {
query = query.Where("type = ?", typ)
}
if err := query.Find(&databaseServer).Error; err != nil {
return nil, err
}
return databaseServer, nil
}
// DatabasesOf 列出单个服务器上的数据库
func (r *databaseRepo) DatabasesOf(ctx context.Context, server *biz.DatabaseServer) ([]*biz.Database, error) {
database := make([]*biz.Database, 0)
switch server.Type {
case biz.DatabaseTypeMongoDB:
mongo, err := db.NewMongoDB(ctx, server.Username, server.Password, fmt.Sprintf("%s:%d", server.Host, server.Port))
if err != nil {
return nil, err
}
if databases, err := mongo.Databases(ctx); err == nil {
for item := range slices.Values(databases) {
database = append(database, &biz.Database{
Type: server.Type,
Name: item.Name,
Server: server.Name,
ServerID: server.ID,
})
}
}
mongo.Close()
case biz.DatabaseTypeSQLite:
sqlite, err := db.NewSQLite(ctx, server.Host)
if err != nil {
return nil, err
}
if tables, err := sqlite.Tables(ctx); err == nil {
for table := range slices.Values(tables) {
database = append(database, &biz.Database{
Type: server.Type,
Name: table,
Server: server.Name,
ServerID: server.ID,
})
}
}
sqlite.Close()
default:
operator, err := r.Operator(ctx, server)
if err != nil {
return nil, err
}
if databases, err := operator.Databases(ctx); err == nil {
for item := range slices.Values(databases) {
database = append(database, &biz.Database{
Type: server.Type,
Name: item.Name,
Server: server.Name,
ServerID: server.ID,
Encoding: item.CharSet,
Comment: item.Comment,
})
}
}
operator.Close()
}
return database, nil
}
// Mongo 构建 MongoDB 客户端
func (r *databaseRepo) Mongo(ctx context.Context, server *biz.DatabaseServer) (*db.MongoDB, error) {
return db.NewMongoDB(ctx, server.Username, server.Password, fmt.Sprintf("%s:%d", server.Host, server.Port))
}
func (r *databaseRepo) Operator(ctx context.Context, server *biz.DatabaseServer) (db.Operator, error) {
switch server.Type {
case biz.DatabaseTypeMysql:
return newMySQLOperator(ctx, server.Username, server.Password, server.Host, server.Port)
case biz.DatabaseTypePostgresql:
postgres, err := db.NewPostgres(ctx, server.Username, server.Password, server.Host, server.Port)
if err != nil {
return nil, err
}
return postgres, nil
case biz.DatabaseTypeClickHouse:
clickhouse, err := db.NewClickHouse(ctx, server.Username, server.Password, fmt.Sprintf("%s:%d", server.Host, server.Port))
if err != nil {
return nil, err
}
return clickhouse, nil
default:
return nil, fmt.Errorf("unsupported database type: %s", server.Type)
}
}