mirror of
https://github.com/Rain-kl/OpenFlare.git
synced 2026-10-05 15:26:36 +08:00
dbaa3bf140
docs(changelog): 修正表述笔误
refactor(cordis): 磁盘缓存改用上上游能力并清理本地副本
按上游/下游归属规约:类型断言守卫已回流 Wavelet(f3d85d5,附回归用例),
本仓库删除 OpenFlare/plugins/server/pkg/cache 整包并改 import 到
Wavelet/pkg/cache/disk,同步后与上游零漂移。
验证:go build 通过;go test ./... exit 0(137 包 ok);256 条路由对拍与
232 条 swagger 操作均零差异;make build-all 四进制;前端零改动。
docs(cordis): 记录 T1 清理结果与五个复用阻塞点
refactor(cordis): server 复用上游 pkg 能力并删除等价本地副本
按上游/下游归属规约清理重复实现,删除 7 个与上游等价的本地包并改 import:
shared/response→pkg/response、pkg/{logger,mail,trace,httppool,cache/ram}→
上游同名包、infra/persistence/batchwriter→pkg/batchwriter。逐项核过差异:
httppool 逐字节相同;logger 的 Config 字段完全一致;response 的 7 个 Abort*
一致;cache/ram 换过去顺带把裸 go 变回带 panic 恢复的 util.Go。
两处非等价差异按语义处理:
- batchwriter.Stats 与 status DTO 原为类型别名,改为消费侧逐字段转换,
避免 model 反向依赖基础设施类型;
- 上游 pkg/idgen 要求显式 Init(本地副本为懒加载自动初始化),本次保留本地
副本,待与 infra 初始化一并迁移(已登记在清理计划)。
验证:go build 通过;go test ./... exit 0(138 包 ok);256 条路由对拍零差异;
make swagger 232 条操作零增减,且归一化后与旧文档深度相等——差异仅为
response.Any / logger.LogEntry 两个定义名随包路径改名,接口形状未变。
chore(cordis): 回流内核与 pkg/util 通用能力并清理 vendoring 污染
按新增的上游/下游归属规约:HandleRaw/BasePath 与版本比较、网络、格式化助手
属通用能力,已提交到 Wavelet 分支 feat/cordis-router-raw-routes,本仓库改为
纯同步获取(pkg/util 已零漂移),补丁登记保留至上游合并。
同时修掉我此前 git add -A 造成的污染:首次 vendoring 把上游工作区里被
gitignore 的运行期产物一起提交进来(upload 的 diskcache 缓存块 650 个与
driver_http/dist 前端构建物 380 个,共 12872 行/1030 文件)。sync-upstream.sh
现显式排除 uploads/dist/data/*.db,.gitignore 补上对应兜底规则。
AGENTS.md 增加上游/下游改动归属规约,并把仍指向前 Cordis 布局的硬性约束
(internal/router + Serve、internal/repository/logstore、internal/platform/bootstrap、
internal/cmd)改到当前插件路径。
验证:go build 通过;go test ./... exit 0(144 包 ok);make swagger 232 条
操作与基线逐条一致;make build-all 四进制;gofmt 干净。
feat(cordis): server 插件化并改由内核挂载控制面路由
新增 plugins/server/plugin.go:Apply 以 ctx.Router().Group(app.api_prefix)
声明根级与 /v1 全部路由;33 个注册函数由 *gin.RouterGroup 改为
core.RouterExtension,RegisterCollection 改用内核新增的 HandleRaw 保留
尾部斜杠变体,AdminMiddlewares 返回 []any(Go 不允许把 []T 展开为 ...any)。
删除 router.Serve 与 registerRoutes,装配根改为 core.App +
driver_http.New(WithEngine(router.BuildEngine())),监听、信号与优雅退出归内核;
前端 SPA 的 NoRoute 兜底因内核暂无贡献点而保留在引擎层。
路由保真证据:plugin_parity_test 对拍 baseline/routes-engine.txt 的 256 条
(方法 路径) 零差异;go test ./... exit 0(144 包 ok,含真实 handler 的
openflare/integration 用例走同一条挂载路径);make swagger 232 条操作与基线
逐条一致;golangci-lint 0 issues;make build-all 四进制;embed_frontend
标签编译通过;前端零改动。
已知待补:带 Redis 的实机 HTTP 冒烟(本机 6379 未启动,session store 与
改造前一样在建店阶段即 fatal),以及 bootstrap 的任务/设置/迁移注册迁入 Apply。
feat(core): RouterExtension 增加 HandleRaw 与 BasePath 以保真尾部斜杠路由
server 插件化的前置:Handle 经 cleanPath 会剥掉尾部斜杠,无法表达
/resource 与 /resource/ 两条不同路由,而 OpenFlare 有 20 个历史 list
端点两者都注册且部署关闭了 RedirectTrailingSlash,缺失即 404。新增
HandleRaw 与 BasePath(作用域包装器同样登记反注册),补 extpoints 用例;
并把 router.Serve 拆出 BuildEngine 以便交给 driver_http.WithEngine 复用,
新增路由表导出 harness,固化 256 条 (方法 路径) 基线供插件化对拍。
上游补丁登记于 backend/OpenFlare/upstream-patches.md,同步脚本改为按目录
前缀输出差异并在同步后提醒确认补丁是否仍在。
验证:go build 通过;go test ./... exit 0(143 包 ok);gofmt 干净。
docs(cordis): 记录 server 插件接入内核的可行路径与内核能力缺口
feat(cordis): agent/relay/flared 落地为内核驱动插件
三个边缘守护进程各新增 plugin.go,实现 core.Plugin + core.Driver
(自定义 DriverType 与同名 profile),装配与生命周期从 main 迁入
Apply/Start/Stop:Apply 负责 JSON 配置加载、运行环境与用户确保、
openresty/frps/frpc 管理器与各服务装配;Start 以 util.Go 拉起阻塞式
runner 与 GeoIP 周期更新;Stop 收敛主循环结果并在超时时报错而非静默。
入口改为 core.NewApp(core.WithProfile(...)) + Prepare/Run,保持
-config 旗标、默认路径、退出码与启动/停止日志不变。
验证:go build 通过;go test ./... exit 0(143 包 ok,含 3 个插件身份
与配置失败路径测试);make build-all 四进制产出;三进制实跑缺失配置
均 exit 1 且错误链保留 load {agent,relay,flared} config 原因;gofmt 干净。
refactor(cordis): 按功能职责拆分为 4 个插件与 share 共享层
backend/OpenFlare 不再平铺遗留分层,改为 plugins/{server,agent,relay,flared}
加 share/:控制面业务(openflare/admin/oauth/user/upload/cap/config/health 与
repository/model/infra/router 等支撑层)归 server;三个边缘守护进程各自成插件;
被两个以上插件消费的 protocol/geoip/wsclient/render/pagesarchive/edge 归 share。
同时把 pkg/util 与 buildinfo 合并回上游 pkg(上游已覆盖全部符号,仅 8 个函数与
2 个类型为 OpenFlare 独有,已一并迁入),装配根统一到 backend/cmd(含三个 daemon
入口),Dockerfile 与 release 工作流的构建路径和 -X 注入路径同步更新。
验证:go build 通过;go test ./... exit 0(141 包 ok);make swagger exit 0 且
232 条 API 操作与基线逐条一致;make build-all 产出 4 进制;-X 注入经二进制
strings 实测生效;日志后端直连门禁改写为按 server 插件业务域扫描并在扫描数为 0
时报错(防门禁静默失效);前端零改动。
feat(cordis): 落地 backend/share 共享层与上游同步脚本
跨插件共享资源(控制消息协议、GeoIP+iputil、边缘守护进程日志)从下游包
移入 backend/share,并声明其只能依赖 core/pkg 与标准/第三方库,禁止反向
引用下游业务与具体插件实现;新增 scripts/sync-upstream.sh 只覆盖
backend/{core,pkg,plugins},同步后 --check 报告零差异,证明与上游逐字一致。
go build 通过,go test ./... exit 0(142 包 ok),前端零改动。
refactor(cordis): 采用与 Wavelet 同构的单模块布局并引入上游内核
按上游结构落位:backend/{core,pkg,plugins} 为 Wavelet 上游拷贝,OpenFlare
全部业务收拢到上游 downstream 所对应的位置 backend/OpenFlare/,模块名保持
Wavelet 以保证上游 import 路径逐字一致、同步零改写;三个 daemon 入口移至
backend/OpenFlare/cmd,backend/cmd 与 main.go 作为控制面装配根。
行为不变:go build 通过,142 个测试包全绿(含上游插件测试),232 条 API
操作与改造前逐条一致,四进制产物正常,前端零改动。swagger 暂只扫描下游代码,
待 P4 挂载上游路由后再纳入 plugins/。
style: 修正模块路径改写导致的 import 分组排序漂移
refactor(layout): Go 代码迁入 backend/ 并将模块名简化为 OpenFlare
对齐上游 Wavelet 的仓库布局,为以第二 module 形态 vendoring Cordis 内核与
平台插件做准备:模块路径整体改写为 OpenFlare,Go 目标加 cd backend,
swaggo 产物移至 backend/docs 并把 json/yaml 复制回 docs/ 供站点消费,
Dockerfile 与 release 工作流的构建目录、ldflags 模块路径同步更新。
行为保持不变:232 条路由与改造前逐条一致,95 个测试包全绿,
四进制产物正常,前端零改动。
chore(cordis): 落地改造计划与 schema/路由基线
新增 legacy_dump_test 迁移快照 harness:在临时 sqlite 库上按生产顺序
(goose.UpTo → zone 导入 → goose.Up)跑完 76 个历史迁移并导出 schema 与
版本序列,作为改造前后一致性门禁的唯一事实来源。同时记录 232 条路由清单
与 foundation 实施计划。
docs(cordis): add OpenFlare Cordis 架构改造设计
明确上游以第二 module 形态 vendoring 进 backend/Wavelet、4 个插件
(server/agent/relay/flared) 全部装载内核,并规定保留 76 个历史 goose
迁移 + 一次性版本 stamp 桥接的迁移方案,配套三方 schema 一致性门禁,
确保已部署库不重跑历史、不丢数据。
545 lines
16 KiB
Go
545 lines
16 KiB
Go
// Copyright 2025 linux.do
|
|
// Copyright 2026 Arctel.net
|
|
// SPDX-License-Identifier: Apache-2.0
|
|
|
|
package system_config
|
|
|
|
import (
|
|
"bufio"
|
|
"bytes"
|
|
"context"
|
|
"encoding/json"
|
|
"net"
|
|
"net/http"
|
|
"net/http/httptest"
|
|
"net/textproto"
|
|
"strings"
|
|
"testing"
|
|
|
|
"Wavelet/OpenFlare/plugins/server/infra/objectstore"
|
|
db "Wavelet/OpenFlare/plugins/server/infra/persistence"
|
|
"Wavelet/OpenFlare/plugins/server/model"
|
|
"Wavelet/OpenFlare/plugins/server/oauth"
|
|
"Wavelet/OpenFlare/plugins/server/repository"
|
|
"Wavelet/OpenFlare/plugins/server/testhelper"
|
|
|
|
"github.com/gin-gonic/gin"
|
|
|
|
"Wavelet/pkg/response"
|
|
)
|
|
|
|
const expectedDefaultConfigsCount = 37
|
|
|
|
func setupTestRouter(authUser *model.User) *gin.Engine {
|
|
r := testhelper.NewTestGinEngine()
|
|
adminGroup := r.Group("/api/v1/admin")
|
|
|
|
// Mock authentication middleware
|
|
adminGroup.Use(func(c *gin.Context) {
|
|
if authUser != nil {
|
|
oauth.SetToContext(c, oauth.UserObjKey, authUser)
|
|
}
|
|
c.Next()
|
|
})
|
|
|
|
adminGroup.POST("/system-configs", CreateSystemConfig)
|
|
adminGroup.GET("/system-configs", ListSystemConfigs)
|
|
|
|
systemConfigRouter := adminGroup.Group("/system-configs/:key")
|
|
{
|
|
systemConfigRouter.GET("", GetSystemConfig)
|
|
systemConfigRouter.PUT("", UpdateSystemConfig)
|
|
}
|
|
|
|
return r
|
|
}
|
|
|
|
func TestCreateSystemConfig(t *testing.T) {
|
|
dbConn, _, cleanup := testhelper.SetupTestEnvironment(t)
|
|
defer cleanup()
|
|
|
|
adminUser := &model.User{ID: 1001, Username: "admin", IsAdmin: true}
|
|
router := setupTestRouter(adminUser)
|
|
|
|
t.Run("create successfully", func(t *testing.T) {
|
|
payload := CreateSystemConfigRequest{
|
|
Key: "custom_key",
|
|
Value: "custom_value",
|
|
Type: "system",
|
|
Visibility: model.ConfigVisibilityVisible,
|
|
Description: "desc",
|
|
}
|
|
body, _ := json.Marshal(payload)
|
|
req, _ := http.NewRequest("POST", "/api/v1/admin/system-configs", bytes.NewBuffer(body))
|
|
req.Header.Set("Content-Type", "application/json")
|
|
w := httptest.NewRecorder()
|
|
router.ServeHTTP(w, req)
|
|
|
|
if w.Code != http.StatusOK {
|
|
t.Fatalf("expected 200 OK, got %d. Body: %s", w.Code, w.Body.String())
|
|
}
|
|
|
|
// Verify database
|
|
var cfg model.SystemConfig
|
|
err := dbConn.Where("key = ?", "custom_key").First(&cfg).Error
|
|
if err != nil {
|
|
t.Fatalf("failed to find system config in DB: %v", err)
|
|
}
|
|
|
|
// Verify caches are invalidated after create and repopulate on read
|
|
_, err = db.Redis.HGet(
|
|
context.Background(),
|
|
db.PrefixedKey(repository.SystemConfigRedisHashKey),
|
|
"custom_key",
|
|
).Result()
|
|
if err == nil {
|
|
t.Fatal("expected redis cache miss immediately after create")
|
|
}
|
|
|
|
loaded, err := repository.GetSystemConfigByKey(context.Background(), "custom_key")
|
|
if err != nil {
|
|
t.Fatalf("GetSystemConfigByKey(custom_key) error = %v", err)
|
|
}
|
|
if loaded.Value != "custom_value" {
|
|
t.Errorf("GetSystemConfigByKey(custom_key).Value = %q, want %q", loaded.Value, "custom_value")
|
|
}
|
|
if loaded.Visibility != model.ConfigVisibilityVisible {
|
|
t.Errorf("GetSystemConfigByKey(custom_key).Visibility = %d, want %d", loaded.Visibility, model.ConfigVisibilityVisible)
|
|
}
|
|
})
|
|
|
|
t.Run("create duplicate key error", func(t *testing.T) {
|
|
// Key "custom_key" already exists from previous test
|
|
payload := CreateSystemConfigRequest{
|
|
Key: "custom_key",
|
|
Value: "another_value",
|
|
Type: "system",
|
|
Description: "desc",
|
|
}
|
|
body, _ := json.Marshal(payload)
|
|
req, _ := http.NewRequest("POST", "/api/v1/admin/system-configs", bytes.NewBuffer(body))
|
|
req.Header.Set("Content-Type", "application/json")
|
|
w := httptest.NewRecorder()
|
|
router.ServeHTTP(w, req)
|
|
|
|
if w.Code != http.StatusBadRequest {
|
|
t.Errorf("expected 400 Bad Request on duplicate key, got %d", w.Code)
|
|
}
|
|
})
|
|
}
|
|
|
|
func TestListSystemConfigs(t *testing.T) {
|
|
_, _, cleanup := testhelper.SetupTestEnvironment(t)
|
|
defer cleanup()
|
|
|
|
adminUser := &model.User{ID: 1001, Username: "admin", IsAdmin: true}
|
|
router := setupTestRouter(adminUser)
|
|
|
|
t.Run("list all seeded configurations", func(t *testing.T) {
|
|
req, _ := http.NewRequest("GET", "/api/v1/admin/system-configs", nil)
|
|
w := httptest.NewRecorder()
|
|
router.ServeHTTP(w, req)
|
|
|
|
if w.Code != http.StatusOK {
|
|
t.Fatalf("expected 200 OK, got %d", w.Code)
|
|
}
|
|
|
|
var resp response.Any
|
|
_ = json.Unmarshal(w.Body.Bytes(), &resp)
|
|
|
|
dataBytes, _ := json.Marshal(resp.Data)
|
|
var configs []model.SystemConfig
|
|
_ = json.Unmarshal(dataBytes, &configs)
|
|
|
|
// Defaults seed configurations
|
|
if len(configs) != expectedDefaultConfigsCount {
|
|
t.Errorf("expected %d default configs, got %d", expectedDefaultConfigsCount, len(configs))
|
|
}
|
|
})
|
|
|
|
t.Run("filter by type business", func(t *testing.T) {
|
|
req, _ := http.NewRequest("GET", "/api/v1/admin/system-configs?type=business", nil)
|
|
w := httptest.NewRecorder()
|
|
router.ServeHTTP(w, req)
|
|
|
|
var resp response.Any
|
|
_ = json.Unmarshal(w.Body.Bytes(), &resp)
|
|
|
|
dataBytes, _ := json.Marshal(resp.Data)
|
|
var configs []model.SystemConfig
|
|
_ = json.Unmarshal(dataBytes, &configs)
|
|
|
|
if len(configs) != 8 {
|
|
t.Errorf("expected 8 business configs, got %d: %v", len(configs), configs)
|
|
}
|
|
})
|
|
}
|
|
|
|
func TestGetSystemConfig(t *testing.T) {
|
|
_, _, cleanup := testhelper.SetupTestEnvironment(t)
|
|
defer cleanup()
|
|
|
|
adminUser := &model.User{ID: 1001, Username: "admin", IsAdmin: true}
|
|
router := setupTestRouter(adminUser)
|
|
|
|
t.Run("get existing configuration", func(t *testing.T) {
|
|
req, _ := http.NewRequest("GET", "/api/v1/admin/system-configs/"+model.ConfigKeySiteName, nil)
|
|
w := httptest.NewRecorder()
|
|
router.ServeHTTP(w, req)
|
|
|
|
if w.Code != http.StatusOK {
|
|
t.Errorf("expected 200 OK, got %d", w.Code)
|
|
}
|
|
|
|
var resp response.Any
|
|
_ = json.Unmarshal(w.Body.Bytes(), &resp)
|
|
|
|
dataBytes, _ := json.Marshal(resp.Data)
|
|
var cfg model.SystemConfig
|
|
_ = json.Unmarshal(dataBytes, &cfg)
|
|
|
|
if cfg.Value != "OpenFlare" {
|
|
t.Errorf("expected 'OpenFlare', got '%s'", cfg.Value)
|
|
}
|
|
})
|
|
|
|
t.Run("get non-existent config", func(t *testing.T) {
|
|
req, _ := http.NewRequest("GET", "/api/v1/admin/system-configs/non_existent_key", nil)
|
|
w := httptest.NewRecorder()
|
|
router.ServeHTTP(w, req)
|
|
|
|
if w.Code != http.StatusNotFound {
|
|
t.Errorf("expected 404 Not Found, got %d", w.Code)
|
|
}
|
|
})
|
|
}
|
|
|
|
func TestUpdateSystemConfig(t *testing.T) {
|
|
dbConn, _, cleanup := testhelper.SetupTestEnvironment(t)
|
|
defer cleanup()
|
|
|
|
adminUser := &model.User{ID: 1001, Username: "admin", IsAdmin: true}
|
|
router := setupTestRouter(adminUser)
|
|
|
|
t.Run("update successfully", func(t *testing.T) {
|
|
hidden := model.ConfigVisibilityHidden
|
|
payload := UpdateSystemConfigRequest{
|
|
Value: "Super Site Name",
|
|
Visibility: &hidden,
|
|
Description: "Updated Description",
|
|
}
|
|
body, _ := json.Marshal(payload)
|
|
req, _ := http.NewRequest("PUT", "/api/v1/admin/system-configs/"+model.ConfigKeySiteName, bytes.NewBuffer(body))
|
|
req.Header.Set("Content-Type", "application/json")
|
|
w := httptest.NewRecorder()
|
|
router.ServeHTTP(w, req)
|
|
|
|
if w.Code != http.StatusOK {
|
|
t.Fatalf("expected 200 OK, got %d. Body: %s", w.Code, w.Body.String())
|
|
}
|
|
|
|
// Verify database
|
|
var cfg model.SystemConfig
|
|
dbConn.Where("key = ?", model.ConfigKeySiteName).First(&cfg)
|
|
if cfg.Value != "Super Site Name" || cfg.Description != "Updated Description" || cfg.Visibility != model.ConfigVisibilityHidden {
|
|
t.Errorf("database values not updated: %+v", cfg)
|
|
}
|
|
|
|
// Verify caches are invalidated after update and repopulate on read
|
|
_, err := db.Redis.HGet(
|
|
context.Background(),
|
|
db.PrefixedKey(repository.SystemConfigRedisHashKey),
|
|
model.ConfigKeySiteName,
|
|
).Result()
|
|
if err == nil {
|
|
t.Fatal("expected redis cache miss immediately after update")
|
|
}
|
|
|
|
loaded, err := repository.GetSystemConfigByKey(context.Background(), model.ConfigKeySiteName)
|
|
if err != nil {
|
|
t.Fatalf("GetSystemConfigByKey(site_name) error = %v", err)
|
|
}
|
|
if loaded.Value != "Super Site Name" {
|
|
t.Errorf("GetSystemConfigByKey(site_name).Value = %q, want %q", loaded.Value, "Super Site Name")
|
|
}
|
|
if loaded.Visibility != model.ConfigVisibilityHidden {
|
|
t.Errorf("GetSystemConfigByKey(site_name).Visibility = %d, want %d", loaded.Visibility, model.ConfigVisibilityHidden)
|
|
}
|
|
})
|
|
|
|
t.Run("update non-existent config", func(t *testing.T) {
|
|
payload := UpdateSystemConfigRequest{
|
|
Value: "New Value",
|
|
}
|
|
body, _ := json.Marshal(payload)
|
|
req, _ := http.NewRequest("PUT", "/api/v1/admin/system-configs/invalid_key", bytes.NewBuffer(body))
|
|
req.Header.Set("Content-Type", "application/json")
|
|
w := httptest.NewRecorder()
|
|
router.ServeHTTP(w, req)
|
|
|
|
if w.Code != http.StatusNotFound {
|
|
t.Errorf("expected 404 Not Found, got %d", w.Code)
|
|
}
|
|
})
|
|
}
|
|
|
|
func TestTestSMTP(t *testing.T) {
|
|
_, _, cleanup := testhelper.SetupTestEnvironment(t)
|
|
defer cleanup()
|
|
|
|
adminUser := &model.User{ID: 1001, Username: "admin", IsAdmin: true}
|
|
r := setupTestRouter(adminUser)
|
|
r.POST("/api/v1/admin/system-configs/smtp/test", TestSMTP)
|
|
|
|
// Start a mock SMTP server
|
|
l, err := net.Listen("tcp", "127.0.0.1:0")
|
|
if err != nil {
|
|
t.Fatalf("failed to start mock smtp server: %v", err)
|
|
}
|
|
defer func() { _ = l.Close() }()
|
|
|
|
port := l.Addr().(*net.TCPAddr).Port
|
|
|
|
go func() {
|
|
conn, err := l.Accept()
|
|
if err != nil {
|
|
return
|
|
}
|
|
defer func() { _ = conn.Close() }()
|
|
|
|
writer := bufio.NewWriter(conn)
|
|
reader := bufio.NewReader(conn)
|
|
tp := textproto.NewReader(reader)
|
|
|
|
// 220 Ready
|
|
_, _ = writer.WriteString("220 mock.smtp.com SMTP Ready\r\n")
|
|
_ = writer.Flush()
|
|
|
|
// Read HELO/EHLO
|
|
_, _ = tp.ReadLine()
|
|
_, _ = writer.WriteString("250-mock.smtp.com\r\n250 AUTH PLAIN\r\n")
|
|
_ = writer.Flush()
|
|
|
|
// Read AUTH PLAIN
|
|
_, _ = tp.ReadLine()
|
|
_, _ = writer.WriteString("235 Authentication successful\r\n")
|
|
_ = writer.Flush()
|
|
|
|
// Read MAIL FROM
|
|
_, _ = tp.ReadLine()
|
|
_, _ = writer.WriteString("250 OK\r\n")
|
|
_ = writer.Flush()
|
|
|
|
// Read RCPT TO
|
|
_, _ = tp.ReadLine()
|
|
_, _ = writer.WriteString("250 OK\r\n")
|
|
_ = writer.Flush()
|
|
|
|
// Read DATA
|
|
_, _ = tp.ReadLine()
|
|
_, _ = writer.WriteString("354 Start mail input\r\n")
|
|
_ = writer.Flush()
|
|
|
|
// Read body lines until dot
|
|
for {
|
|
line, err := tp.ReadLine()
|
|
if err != nil || line == "." {
|
|
break
|
|
}
|
|
}
|
|
_, _ = writer.WriteString("250 OK\r\n")
|
|
_ = writer.Flush()
|
|
|
|
// Read QUIT
|
|
_, _ = tp.ReadLine()
|
|
_, _ = writer.WriteString("221 Bye\r\n")
|
|
_ = writer.Flush()
|
|
}()
|
|
|
|
payload := TestSMTPRequest{
|
|
SMTPHost: "127.0.0.1",
|
|
SMTPPort: port,
|
|
SMTPUsername: "sender@example.com",
|
|
SMTPPassword: "password",
|
|
To: "recipient@example.com",
|
|
}
|
|
|
|
body, _ := json.Marshal(payload)
|
|
req, _ := http.NewRequest("POST", "/api/v1/admin/system-configs/smtp/test", bytes.NewBuffer(body))
|
|
req.Header.Set("Content-Type", "application/json")
|
|
w := httptest.NewRecorder()
|
|
r.ServeHTTP(w, req)
|
|
|
|
if w.Code != http.StatusOK {
|
|
t.Fatalf("expected 200 OK, got %d. Body: %s", w.Code, w.Body.String())
|
|
}
|
|
|
|
var resp response.Any
|
|
json.Unmarshal(w.Body.Bytes(), &resp)
|
|
|
|
dataBytes, _ := json.Marshal(resp.Data)
|
|
var testResp TestSMTPResponse
|
|
json.Unmarshal(dataBytes, &testResp)
|
|
|
|
if !testResp.Success {
|
|
t.Errorf("expected test success, got failed: %s. Log: %s", testResp.Error, testResp.Log)
|
|
}
|
|
}
|
|
|
|
func TestUpdateStorageConfigValidation(t *testing.T) {
|
|
dbConn, _, cleanup := testhelper.SetupTestEnvironment(t)
|
|
defer cleanup()
|
|
|
|
adminUser := &model.User{ID: 1001, Username: "admin", IsAdmin: true}
|
|
router := setupTestRouter(adminUser)
|
|
|
|
t.Run("update storage config successfully", func(t *testing.T) {
|
|
tempDir := t.TempDir()
|
|
cfg := objectstore.DefaultConfig()
|
|
cfg.Local.Root = tempDir
|
|
|
|
cfgBytes, _ := json.Marshal(cfg)
|
|
payload := UpdateSystemConfigRequest{
|
|
Value: string(cfgBytes),
|
|
}
|
|
body, _ := json.Marshal(payload)
|
|
req, _ := http.NewRequest("PUT", "/api/v1/admin/system-configs/storage_config", bytes.NewBuffer(body))
|
|
req.Header.Set("Content-Type", "application/json")
|
|
w := httptest.NewRecorder()
|
|
router.ServeHTTP(w, req)
|
|
|
|
if w.Code != http.StatusOK {
|
|
t.Fatalf("expected 200 OK, got %d. Body: %s", w.Code, w.Body.String())
|
|
}
|
|
|
|
// Verify database
|
|
var dbCfg model.SystemConfig
|
|
dbConn.Where("key = ?", "storage_config").First(&dbCfg)
|
|
var savedCfg objectstore.Config
|
|
_ = json.Unmarshal([]byte(dbCfg.Value), &savedCfg)
|
|
if savedCfg.Local.Root != tempDir {
|
|
t.Errorf("expected local root to be updated to %s, got %s", tempDir, savedCfg.Local.Root)
|
|
}
|
|
})
|
|
|
|
t.Run("update storage config failed connectivity check", func(t *testing.T) {
|
|
cfg := objectstore.DefaultConfig()
|
|
cfg.Driver = objectstore.DriverS3
|
|
cfg.S3.Bucket = "non-existent-bucket"
|
|
cfg.S3.Endpoint = "http://127.0.0.1:9999" // Will fail connectivity check
|
|
|
|
cfgBytes, _ := json.Marshal(cfg)
|
|
payload := UpdateSystemConfigRequest{
|
|
Value: string(cfgBytes),
|
|
}
|
|
body, _ := json.Marshal(payload)
|
|
req, _ := http.NewRequest("PUT", "/api/v1/admin/system-configs/storage_config", bytes.NewBuffer(body))
|
|
req.Header.Set("Content-Type", "application/json")
|
|
w := httptest.NewRecorder()
|
|
router.ServeHTTP(w, req)
|
|
|
|
if w.Code != http.StatusBadRequest {
|
|
t.Errorf("expected 400 Bad Request, got %d. Body: %s", w.Code, w.Body.String())
|
|
}
|
|
})
|
|
|
|
t.Run("reject driver switch when uploads exist", func(t *testing.T) {
|
|
upload := model.Upload{
|
|
ID: 88001,
|
|
UserID: 1,
|
|
FileName: "keep.txt",
|
|
FilePath: "uploads/keep.txt",
|
|
FileSize: 4,
|
|
MimeType: "text/plain",
|
|
Extension: "txt",
|
|
Type: "attachment",
|
|
Status: model.UploadStatusUsed,
|
|
}
|
|
if err := dbConn.Create(&upload).Error; err != nil {
|
|
t.Fatalf("seed upload failed: %v", err)
|
|
}
|
|
|
|
tempDir := t.TempDir()
|
|
cfg := objectstore.DefaultConfig()
|
|
cfg.Driver = objectstore.DriverS3
|
|
cfg.S3.Endpoint = "http://127.0.0.1:19998"
|
|
cfg.S3.Region = "us-east-1"
|
|
cfg.S3.Bucket = "wavelet"
|
|
cfg.S3.AccessKeyID = "test"
|
|
cfg.S3.SecretAccessKey = "test"
|
|
cfg.Local.Root = tempDir
|
|
|
|
cfgBytes, _ := json.Marshal(cfg)
|
|
payload := UpdateSystemConfigRequest{Value: string(cfgBytes)}
|
|
body, _ := json.Marshal(payload)
|
|
req, _ := http.NewRequest("PUT", "/api/v1/admin/system-configs/storage_config", bytes.NewBuffer(body))
|
|
req.Header.Set("Content-Type", "application/json")
|
|
w := httptest.NewRecorder()
|
|
router.ServeHTTP(w, req)
|
|
|
|
if w.Code != http.StatusBadRequest {
|
|
t.Fatalf("expected 400 Bad Request, got %d. Body: %s", w.Code, w.Body.String())
|
|
}
|
|
if !strings.Contains(w.Body.String(), StorageDriverSwitchRequiresMigration) {
|
|
t.Fatalf("expected migration-required error, got: %s", w.Body.String())
|
|
}
|
|
})
|
|
|
|
t.Run("switch to local while active s3 is unreachable", func(t *testing.T) {
|
|
if err := dbConn.Where("1 = 1").Delete(&model.Upload{}).Error; err != nil {
|
|
t.Fatalf("clear uploads failed: %v", err)
|
|
}
|
|
|
|
activeCfg := objectstore.DefaultConfig()
|
|
activeCfg.Driver = objectstore.DriverS3
|
|
activeCfg.S3.Endpoint = "http://127.0.0.1:9999"
|
|
activeCfg.S3.Region = "us-east-1"
|
|
activeCfg.S3.Bucket = "wavelet"
|
|
activeCfg.S3.AccessKeyID = "test"
|
|
activeCfg.S3.SecretAccessKey = "test"
|
|
activeBytes, _ := json.Marshal(activeCfg)
|
|
seedCfg := model.SystemConfig{
|
|
Key: "storage_config",
|
|
Value: string(activeBytes),
|
|
Type: "system",
|
|
}
|
|
if err := dbConn.Where("key = ?", "storage_config").
|
|
Assign(map[string]any{"value": seedCfg.Value, "type": seedCfg.Type}).
|
|
FirstOrCreate(&seedCfg).Error; err != nil {
|
|
t.Fatalf("seed active storage config failed: %v", err)
|
|
}
|
|
|
|
tempDir := t.TempDir()
|
|
stagedCfg := activeCfg
|
|
stagedCfg.Driver = objectstore.DriverLocal
|
|
stagedCfg.Local.Root = tempDir
|
|
|
|
cfgBytes, _ := json.Marshal(stagedCfg)
|
|
payload := UpdateSystemConfigRequest{Value: string(cfgBytes)}
|
|
body, _ := json.Marshal(payload)
|
|
req, _ := http.NewRequest("PUT", "/api/v1/admin/system-configs/storage_config", bytes.NewBuffer(body))
|
|
req.Header.Set("Content-Type", "application/json")
|
|
w := httptest.NewRecorder()
|
|
router.ServeHTTP(w, req)
|
|
|
|
if w.Code != http.StatusOK {
|
|
t.Fatalf("expected 200 OK, got %d. Body: %s", w.Code, w.Body.String())
|
|
}
|
|
|
|
var dbCfg model.SystemConfig
|
|
if err := dbConn.Where("key = ?", "storage_config").First(&dbCfg).Error; err != nil {
|
|
t.Fatalf("load saved storage config failed: %v", err)
|
|
}
|
|
var savedCfg objectstore.Config
|
|
if err := json.Unmarshal([]byte(dbCfg.Value), &savedCfg); err != nil {
|
|
t.Fatalf("parse saved storage config failed: %v", err)
|
|
}
|
|
if savedCfg.Driver != objectstore.DriverLocal {
|
|
t.Fatalf("active driver = %q, want %q after save", savedCfg.Driver, objectstore.DriverLocal)
|
|
}
|
|
if savedCfg.Local.Root != tempDir {
|
|
t.Fatalf("staged local root = %q, want %q", savedCfg.Local.Root, tempDir)
|
|
}
|
|
})
|
|
}
|