fix(v1): разблокировать сборку, починить жизненный цикл cron и закрыть каналы утечки

Сборка не собиралась: два контракта приёмки роняли её на корректном коде. verify_api_namespace_contract искал возвращение legacy-пространства имён через grep по '/hui' и находил router_test.go, который ПЕРЕЧИСЛЯЕТ этот префикс, чтобы доказать отсутствие маршрута, и сам versions.sh, где строка стоит в тексте проверки. Падение приходило шестым шагом из четырнадцати, до резолва Hysteria. За ним прятался второй такой же: проверка транзакционности импорта пиров брала файл от начала applyPeerImportEntry и до конца, захватывая объявленные ниже ExistPeerName и UpdatePeerLastConnectionAt. Обе проверки теперь смотрят на код, а не на упоминания: добавлены помощники code_without_comments и code_mentions_in, а отсутствие legacy-маршрута доказывает тест на таблице маршрутов собранного роутера. Планировщик стал собственностью процесса. InitCron вызывался из runServer и на каждом вызове создавал новый cron.New(), не сохраняя ссылку; cron.Stop() не вызывался нигде. Смена RESET_TRAFFIC_CRON выполняла StopServer(), точка входа крутила for { runServer() } — и каждая правка добавляла целый дублирующий набор джоб, а старое расписание сброса продолжало работать. Фиксированные джобы регистрируются один раз, расписание переносится на месте по EntryID, HTTP-сервер не трогается. Добавлено штатное завершение по SIGTERM. Выражение проверяется до записи в базу тем же парсером (cron.ParseStandard), которым его разбирает планировщик: раньше невалидная строка сохранялась, API отвечал успехом, а сброс трафика молча исчезал. updateConfigs стал атомарным: полная проверка партии, одна транзакция, применение к рантайму. Прежний тест ставил запрещённый ключ первым и не смотрел в базу — поймать частичное применение он был неспособен. Удалены четыре ключа таблицы config без единого потребителя: HYSTERIA2_ENABLE, HYSTERIA2_CONFIG (второй источник истины, читался первым), HYSTERIA2_TRAFFIC_TIME и HYSTERIA2_CONFIG_REMARK. Имя профиля в share URI выводится из имени пира. Безопасность: - bootstrap-пароль администратора больше не генерируется и не пишется в журнал, который отдаётся кнопкой выгрузки; отсутствие env — отказ старта; - собственный журнал админки санитизируется наравне с чужим; - golang-jwt/jwt v3 -> v5: GO-2025-3553 не имеет исправленной версии в v3 и достижима с неаутентифицированного запроса; набор алгоритмов подписи зафиксирован через WithValidMethods; - удалён вход по несолёному SHA-224 из предыдущего поколения; - убран modulo bias в util.RandomString — единственном генераторе секретов; - пир установщика защищён во всех путях записи, а не только в импорте; - удалена латентная паника в service.GetToken и недостижимая ветка GetAdminInfo, проверявшая меньше, чем middleware. Toolchain: Go 1.21.13 -> 1.26.7, Node 20.19.0 (EOL) -> 24.20.0. На прежнем графе govulncheck находил 21 вызываемую уязвимость, 17 из них в stdlib, попадающей в production-бинарь. Сейчас — ноль. Добавлен обязательный шаг проверки зависимостей (govulncheck + pnpm audit) с записью результата в metadata пакета.
2026-08-29 21:37:38 +05:00
parent 672d455467
commit b99be7d514
50 changed files with 3907 additions and 1622 deletions
@@ -18,14 +18,100 @@ func SaveConfig(config entity.Config) (int64, error) {
 }

 func UpdateConfig(keys []string, updates map[string]interface{}) error {
-	if len(updates) > 0 {
-		updates["update_time"] = time.Now().Format("2006-01-02 15:04:05")
-		if tx := sqliteDB.Model(&entity.Config{}).
-			Where("key in ?", keys).
-			Updates(updates); tx.Error != nil {
-			logrus.Errorf("%v", tx.Error)
-			return errors.New(constant.SysError)
-		}
+	return updateConfigOn(sqliteDB, keys, updates)
+}
+
+// UpsertConfigValue записывает значение ключа, создавая строку, если её нет.
+//
+// Заменяет две зеркальные самодельные схемы, которые жили в sqlite.go и обе
+// были неверны:
+//
+//  1. «сначала INSERT, при ошибке UPDATE» — работало, но на КАЖДОЙ первой
+//     загрузке печатало в журнал `duplicated key not allowed` уровня error, по
+//     одной строке на секрет. Оператор видел три ошибки на совершенно здоровом
+//     старте и не имел способа отличить их от настоящих;
+//
+//  2. «сначала UPDATE, при ошибке INSERT» — тихо не делало НИЧЕГО, если строки
+//     не было. UPDATE без совпавших строк не является ошибкой: gorm возвращает
+//     nil и RowsAffected = 0, поэтому ветка INSERT не выполнялась, а вызывающий
+//     получал сгенерированный секрет как успешно сохранённый. Для JWT_SECRET
+//     это означало бы подпись токенов ключом, которого нет в базе. Спасала
+//     только предварительная вставка пустых строк в seedBaseConfig — то есть
+//     корректность держалась на порядке выполнения миграций.
+//
+// Здесь решение принимается по RowsAffected, а не по наличию ошибки.
+func UpsertConfigValue(key string, value string) error {
+	if sqliteDB == nil {
+		return errors.New("sqlite is not initialized")
+	}
+
+	tx := sqliteDB.Model(&entity.Config{}).
+		Where("key = ?", key).
+		Updates(map[string]interface{}{
+			"value":       value,
+			"update_time": time.Now().Format("2006-01-02 15:04:05"),
+		})
+	if tx.Error != nil {
+		logrus.Errorf("%v", tx.Error)
+		return errors.New(constant.SysError)
+	}
+	if tx.RowsAffected > 0 {
+		return nil
+	}
+
+	remark := key
+	if _, err := SaveConfig(entity.Config{Key: &key, Value: &value, Remark: &remark}); err != nil {
+		return err
+	}
+	return nil
+}
+
+// ConfigTx — записи в таблицу `config` в границах одной единицы работы.
+//
+// Интерфейс повторяет PeerTx и по той же причине: транзакция должна быть
+// доступна слою service, а *gorm.DB туда протекать не должен. Реализация одна
+// и та же для глобального соединения и для транзакции, поэтому «обычный» и
+// «транзакционный» пути не могут разойтись в поведении.
+type ConfigTx interface {
+	UpdateConfig(keys []string, updates map[string]interface{}) error
+}
+
+type configTx struct {
+	db *gorm.DB
+}
+
+// WithConfigTx выполняет fn в одной транзакции: либо применяются все записи
+// партии, либо ни одной.
+//
+// Что было. updateConfigs валидировал и записывал настройки в одном цикле:
+// проверил элемент — записал, проверил следующий. Партия из разрешённого
+// ключа и запрещённого применяла первый и возвращала ошибку на втором, то есть
+// оператор получал отказ на запрос, который систему уже изменил. Тест на этот
+// случай существовал, но ставил запрещённый ключ ПЕРВЫМ и не смотрел в базу,
+// поэтому поймать частичное применение был неспособен по построению.
+func WithConfigTx(fn func(tx ConfigTx) error) error {
+	if sqliteDB == nil {
+		return errors.New("sqlite is not initialized")
+	}
+	return sqliteDB.Transaction(func(tx *gorm.DB) error {
+		return fn(configTx{db: tx})
+	})
+}
+
+func (c configTx) UpdateConfig(keys []string, updates map[string]interface{}) error {
+	return updateConfigOn(c.db, keys, updates)
+}
+
+func updateConfigOn(db *gorm.DB, keys []string, updates map[string]interface{}) error {
+	if len(updates) == 0 {
+		return nil
+	}
+	updates["update_time"] = time.Now().Format("2006-01-02 15:04:05")
+	if tx := db.Model(&entity.Config{}).
+		Where("key in ?", keys).
+		Updates(updates); tx.Error != nil {
+		logrus.Errorf("%v", tx.Error)
+		return errors.New(constant.SysError)
 	}
 	return nil
 }
@@ -95,6 +95,22 @@ func envBoolAsInt(name string, fallback int) int {
 	return fallback
 }

+// ensureSecureBootstrapAdmin создаёт первую учётную запись администратора.
+//
+// Пароль приходит ТОЛЬКО из HY2XS_ADMIN_INITIAL_PASSWORD, и его отсутствие —
+// отказ старта, а не повод придумать пароль самостоятельно.
+//
+// Что было. При пустом env админка генерировала пароль сама и печатала его
+// двумя logrus.Warnf — то есть открытым текстом в /var/log/hy2xs/hy2xs-admin.log.
+// Этот же файл отдаётся оператору кнопкой выгрузки журнала и попадает в
+// diagnostics-бандл, который люди пересылают в переписке. Получался постоянный
+// канал утечки учётных данных администратора.
+//
+// Второе, менее очевидное: сгенерированный здесь пароль не знает НИКТО, кроме
+// журнала. Оркестратор пишет свой в /etc/hy2xs/bootstrap-admin.secret и в
+// hy2xs.env, и именно он считается контрактом установки. Значит, попадание
+// сюда означает не «нужно что-то придумать», а повреждённый контракт запуска —
+// и правильная реакция на него громкая, а не молчаливая.
 func ensureSecureBootstrapAdmin() error {
 	adminUser := strings.TrimSpace(os.Getenv("HY2XS_ADMIN_USER"))
 	if adminUser == "" {
@@ -105,14 +121,16 @@ func ensureSecureBootstrapAdmin() error {
 	}

 	adminPassword := strings.TrimSpace(os.Getenv("HY2XS_ADMIN_INITIAL_PASSWORD"))
-	passwordGenerated := false
 	if adminPassword == "" {
-		password, pwdErr := util.RandomString(18)
-		if pwdErr != nil {
-			return pwdErr
-		}
-		adminPassword = password
-		passwordGenerated = true
+		return errors.New(
+			"HY2XS_ADMIN_INITIAL_PASSWORD не задан, а учётной записи администратора в базе нет.\n" +
+				"Это признак повреждённого контракта запуска: пароль первого администратора\n" +
+				"задаёт install-оркестратор через /etc/hy2xs/hy2xs.env, а копию кладёт в\n" +
+				"/etc/hy2xs/bootstrap-admin.secret. Придумывать его самостоятельно админка не\n" +
+				"будет: такой пароль не знал бы никто, кроме журнала.\n" +
+				"Проверьте EnvironmentFile в юните hy2xs-admin и запустите\n" +
+				"`hy2xs-orchestrator repair --allow-partial-state`.",
+		)
 	}
 	forcePasswordChange := envBoolAsInt("HY2XS_FORCE_PASSWORD_CHANGE", 1)
 	status := int64(1)
@@ -135,23 +153,22 @@ func ensureSecureBootstrapAdmin() error {
 	if _, saveErr := SaveAdminUser(account); saveErr != nil {
 		return saveErr
 	}
-	if passwordGenerated {
-		logrus.Warnf("Initial admin username: %s", adminUser)
-		logrus.Warnf("Initial admin password: %s", adminPassword)
-	}
 	return nil
 }

+// ensureSecureBootstrapPeer создаёт пир установщика.
+//
+// Секрет приходит ТОЛЬКО из HY2XS_ADMIN_CON_PASS по той же причине, что и
+// пароль администратора, и с дополнительным следствием: этот секрет лежит ещё
+// и в /etc/hy2xs/bootstrap-admin.secret, откуда его читает smoke оркестратора
+// для проверки machine-auth. Сгенерированный здесь секрет разошёлся бы с
+// файлом, и первая же проверка подключения после установки провалилась бы —
+// на корректном во всём остальном сервере.
+//
+// Порядок проверок здесь важен: сначала выясняем, нужно ли вообще создавать
+// пир, и только потом требуем env. Иначе перезапуск уже установленного сервиса
+// без переменной падал бы на ровном месте.
 func ensureSecureBootstrapPeer() error {
-	bootstrapSecret := strings.TrimSpace(os.Getenv("HY2XS_ADMIN_CON_PASS"))
-	if bootstrapSecret == "" {
-		generated, err := util.RandomString(24)
-		if err != nil {
-			return err
-		}
-		bootstrapSecret = generated
-	}
-
 	if _, err := GetPeer("auth_id = ?", "bootstrap-admin-peer"); err == nil {
 		return nil
 	}
@@ -159,6 +176,19 @@ func ensureSecureBootstrapPeer() error {
 		return nil
 	}

+	bootstrapSecret := strings.TrimSpace(os.Getenv("HY2XS_ADMIN_CON_PASS"))
+	if bootstrapSecret == "" {
+		return errors.New(
+			"HY2XS_ADMIN_CON_PASS не задан, а пира установщика bootstrap-admin-peer в базе нет.\n" +
+				"Секрет этого пира задаёт install-оркестратор и кладёт копию в\n" +
+				"/etc/hy2xs/bootstrap-admin.secret, откуда его читает проверка machine-auth.\n" +
+				"Секрет, придуманный админкой, разошёлся бы с этим файлом, и проверка\n" +
+				"подключения после установки провалилась бы.\n" +
+				"Проверьте EnvironmentFile в юните hy2xs-admin и запустите\n" +
+				"`hy2xs-orchestrator repair --allow-partial-state`.",
+		)
+	}
+
 	secretDigest, err := buildPeerSecretDigest(bootstrapSecret)
 	if err != nil {
 		return err
@@ -205,15 +235,23 @@ func ensureSecureBootstrapPeer() error {
 }

 func buildPeerSecretDigest(rawSecret string) (string, error) {
-	secretKey, err := getOrCreatePeerSecretDigestKey()
+	secretKey, err := GetOrCreatePeerSecretDigestKey()
 	if err != nil {
 		return "", err
 	}
 	return util.HmacSHA256Hex(rawSecret, secretKey), nil
 }

-func getOrCreatePeerSecretEncryptionKey() ([]byte, error) {
-	raw, err := getOrCreateConfigKey(constant.PeerSecretEncryptionKey, 32)
+// GetOrCreatePeerSecretEncryptionKey возвращает master-key шифрования секретов
+// пиров, при необходимости создавая его.
+//
+// Экспортирована, чтобы у ключа была ровно одна реализация. Раньше идентичная
+// копия жила в service/peer_secret.go: две функции с одинаковым именем в разных
+// пакетах, обе создающие один и тот же материал шифрования. Расхождение между
+// ними означало бы, что секреты пиров шифруются одним ключом, а
+// расшифровываются другим, и обнаружилось бы это на живых пирах.
+func GetOrCreatePeerSecretEncryptionKey() ([]byte, error) {
+	raw, err := GetOrCreateConfigSecret(constant.PeerSecretEncryptionKey, 32)
 	if err != nil {
 		return nil, err
 	}
@@ -221,41 +259,42 @@ func getOrCreatePeerSecretEncryptionKey() ([]byte, error) {
 	if decErr == nil {
 		return decoded, nil
 	}
+
+	// Ключ, записанный не в base64: приводим к стабильному представлению один
+	// раз, сохранив исходные байты, — иначе уже зашифрованные секреты пиров
+	// перестали бы расшифровываться.
 	plain := []byte(strings.TrimSpace(raw))
 	if len(plain) < 32 {
 		return nil, errors.New("invalid peer secret encryption key")
 	}
 	plain = plain[:32]
-	encoded := base64.StdEncoding.EncodeToString(plain)
-	if err := UpdateConfig([]string{constant.PeerSecretEncryptionKey}, map[string]interface{}{"value": encoded}); err != nil {
+	if err := UpsertConfigValue(constant.PeerSecretEncryptionKey, base64.StdEncoding.EncodeToString(plain)); err != nil {
 		return nil, err
 	}
 	return plain, nil
 }

 func encryptBootstrapPeerSecret(rawSecret string) (string, error) {
-	key, err := getOrCreatePeerSecretEncryptionKey()
+	key, err := GetOrCreatePeerSecretEncryptionKey()
 	if err != nil {
 		return "", err
 	}
 	return util.EncryptAESGCM(rawSecret, key)
 }

-func getOrCreateConfigKey(key string, size int) (string, error) {
+// GetOrCreateConfigSecret читает секрет из таблицы `config`, создавая его при
+// первом обращении. Единственная реализация этой операции в продукте.
+func GetOrCreateConfigSecret(key string, size int) (string, error) {
 	cfg, err := GetConfig("key = ?", key)
 	if err == nil && cfg.Value != nil && strings.TrimSpace(*cfg.Value) != "" {
 		return strings.TrimSpace(*cfg.Value), nil
 	}
-	raw, genErr := util.RandomString(size)
+	value, genErr := util.RandomString(size)
 	if genErr != nil {
 		return "", genErr
 	}
-	value := raw
-	remark := key
-	if _, saveErr := SaveConfig(entity.Config{Key: &key, Value: &value, Remark: &remark}); saveErr != nil {
-		if updErr := UpdateConfig([]string{key}, map[string]interface{}{"value": value}); updErr != nil {
-			return "", updErr
-		}
+	if err := UpsertConfigValue(key, value); err != nil {
+		return "", err
 	}
 	return value, nil
 }
@@ -300,6 +339,7 @@ func runMigrations() error {
 		{version: "001_admin_peer_split", apply: migrateAdminPeerSplit},
 		{version: "004_traffic_samples_and_aggregates", apply: migrateTrafficTables},
 		{version: "005_metric_sample", apply: migrateMetricSampleTable},
+		{version: "006_drop_dead_config_keys", apply: migrateDropDeadConfigKeys},
 	}

 	for _, m := range migrations {
@@ -322,12 +362,16 @@ func runMigrations() error {

 func seedBaseConfig() error {
 	// Сетевых настроек панели здесь нет: ими владеет оркестратор.
+	//
+	// HYSTERIA2_ENABLE, HYSTERIA2_CONFIG, HYSTERIA2_TRAFFIC_TIME и
+	// HYSTERIA2_CONFIG_REMARK удалены: ни один из них не имел потребителя в
+	// рантайме, см. model/constant/config.go. Строки этих ключей на уже
+	// установленных машинах убирает миграция 006_drop_dead_config_keys.
+	//
+	// Оставшееся — четыре внутренних секрета и единственная настройка, которой
+	// действительно владеет панель.
 	defaults := map[string]string{
 		constant.JwtSecret:                   "",
-		constant.Hysteria2Enable:             "0",
-		constant.Hysteria2Config:             "",
-		constant.Hysteria2TrafficTime:        "10",
-		constant.Hysteria2ConfigRemark:       "",
 		constant.ResetTrafficCron:            "0 0 * * *",
 		constant.Hysteria2TrafficStatsSecret: "",
 		constant.PeerSecretKey:               "",
@@ -363,6 +407,34 @@ func migrateBaseConfig() error {
 	return seedBaseConfig()
 }

+// migrateDropDeadConfigKeys убирает из таблицы `config` ключи, у которых не
+// осталось ни одного потребителя.
+//
+// Миграция нужна не ради места в базе, а ради того же инварианта, что и
+// allowlist: строка в таблице секретов, которую никто не читает и не пишет,
+// рано или поздно кем-нибудь прочитается. Ключи перечислены здесь строковыми
+// литералами намеренно — соответствующих констант в коде уже нет, и появиться
+// они не должны.
+//
+// Миграция безопасна на чистой установке (удалять нечего) и идемпотентна:
+// применяется один раз и отмечается в schema_migrations. Номера предыдущих
+// миграций не тронуты — они уже записаны на установленных машинах.
+func migrateDropDeadConfigKeys() error {
+	deadKeys := []string{
+		"HYSTERIA2_ENABLE",
+		"HYSTERIA2_CONFIG",
+		"HYSTERIA2_TRAFFIC_TIME",
+		"HYSTERIA2_CONFIG_REMARK",
+	}
+	for _, key := range deadKeys {
+		if tx := sqliteDB.Exec("DELETE FROM config WHERE key = ?", key); tx.Error != nil {
+			logrus.Errorf("sqlite migration exec err: %v", tx.Error)
+			return errors.New("sqlite migration exec err")
+		}
+	}
+	return nil
+}
+
 func migrationApplied(version string) (bool, error) {
 	var count int64
 	if tx := sqliteDB.Raw("SELECT COUNT(1) FROM schema_migrations WHERE version = ?", version).Scan(&count); tx.Error != nil {
@@ -428,34 +500,17 @@ func getOrCreateJwtSecret() (string, error) {
 	if secErr != nil {
 		return "", secErr
 	}
-	if err := UpdateConfig([]string{constant.JwtSecret}, map[string]interface{}{"value": secret}); err != nil {
-		key := constant.JwtSecret
-		remark := "JWT signing secret"
-		if _, saveErr := SaveConfig(entity.Config{Key: &key, Value: &secret, Remark: &remark}); saveErr != nil {
-			return "", saveErr
-		}
+	if err := UpsertConfigValue(constant.JwtSecret, secret); err != nil {
+		return "", err
 	}
 	return secret, nil
 }

-func getOrCreatePeerSecretDigestKey() (string, error) {
-	if existing, err := GetConfig("key = ?", constant.PeerSecretKey); err == nil {
-		if existing.Value != nil && strings.TrimSpace(*existing.Value) != "" {
-			return strings.TrimSpace(*existing.Value), nil
-		}
-	}
-	keyValue, keyErr := util.RandomString(48)
-	if keyErr != nil {
-		return "", keyErr
-	}
-	if err := UpdateConfig([]string{constant.PeerSecretKey}, map[string]interface{}{"value": keyValue}); err != nil {
-		key := constant.PeerSecretKey
-		remark := "Peer secret digest key"
-		if _, saveErr := SaveConfig(entity.Config{Key: &key, Value: &keyValue, Remark: &remark}); saveErr != nil {
-			return "", saveErr
-		}
-	}
-	return keyValue, nil
+// GetOrCreatePeerSecretDigestKey — HMAC-ключ, которым считается secret_digest
+// пира. Экспортирована по той же причине, что и ключ шифрования: копия этой же
+// логики жила в service/peer_secret.go под именем GetPeerSecretKey.
+func GetOrCreatePeerSecretDigestKey() (string, error) {
+	return GetOrCreateConfigSecret(constant.PeerSecretKey, 48)
 }

 func migrateTrafficTables() error {
@@ -542,43 +597,33 @@ func tableExists(tableName string) bool {
 	}
 	return count > 0
 }
+// ensureTrafficStatsSecret синхронизирует machine token с окружением.
+//
+// Владелец значения — оркестратор: он же кладёт этот секрет в
+// /etc/hysteria/config.yaml, откуда Hysteria берёт его для обращения к
+// /internal/hysteria/auth. Поэтому env всегда побеждает содержимое базы, а
+// собственная генерация — только для случая, когда переменной нет вовсе.
+//
+// Четыре почти одинаковые ветки INSERT/UPDATE свёрнуты в UpsertConfigValue:
+// каждая из них по-своему решала «есть строка или нет», и две из четырёх
+// решали неверно (см. комментарий к UpsertConfigValue).
 func ensureTrafficStatsSecret() error {
 	envSecret := strings.TrimSpace(os.Getenv("HY2XS_HYSTERIA_TRAFFIC_STATS_SECRET"))
 	if envSecret != "" {
-		if existing, err := GetConfig("key = ?", constant.Hysteria2TrafficStatsSecret); err == nil {
-			if existing.Value != nil && strings.TrimSpace(*existing.Value) != envSecret {
-				return UpdateConfig([]string{constant.Hysteria2TrafficStatsSecret}, map[string]interface{}{"value": envSecret})
-			}
-			return nil
-		}
-		key := constant.Hysteria2TrafficStatsSecret
-		remark := "Hysteria2 trafficStats API secret"
-		if _, saveErr := SaveConfig(entity.Config{Key: &key, Value: &envSecret, Remark: &remark}); saveErr != nil {
-			return saveErr
-		}
-		return nil
+		return UpsertConfigValue(constant.Hysteria2TrafficStatsSecret, envSecret)
 	}

 	if existing, err := GetConfig("key = ?", constant.Hysteria2TrafficStatsSecret); err == nil {
 		if existing.Value != nil && strings.TrimSpace(*existing.Value) != "" {
 			return nil
 		}
-		secret, secErr := util.RandomString(32)
-		if secErr != nil {
-			return secErr
-		}
-		return UpdateConfig([]string{constant.Hysteria2TrafficStatsSecret}, map[string]interface{}{"value": secret})
 	}
+
 	secret, secErr := util.RandomString(32)
 	if secErr != nil {
 		return secErr
 	}
-	key := constant.Hysteria2TrafficStatsSecret
-	remark := "Hysteria2 trafficStats API secret"
-	if _, saveErr := SaveConfig(entity.Config{Key: &key, Value: &secret, Remark: &remark}); saveErr != nil {
-		return saveErr
-	}
-	return nil
+	return UpsertConfigValue(constant.Hysteria2TrafficStatsSecret, secret)
 }

 func CloseSqliteDB() error {