Строчные ссылки на файлы модов и своё ожидание готовности
Игра приводит путь к вложенным файлам анимаций в нижний регистр целиком, вместе с именем папки мода. На Linux такие файлы не находятся, и сервер сыплет в лог сотнями FileNotFoundException. Кнопка на вкладке «Моды» раскладывает рядом строчные симлинки: файлы модов не меняются, повторный запуск после докачки безопасен, а на файловых системах без различия регистра кнопка честно говорит, что делать нечего. Ожидание строки о готовности стало настройкой профиля со значением по умолчанию 30 минут вместо общих 15. Сборке на три сотни модов пятнадцати не хватает, и панель объявляла бы работающим сервер, который ещё никого не пустит. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This commit is contained in:
@@ -5,16 +5,17 @@ import (
|
||||
"io"
|
||||
"net/http"
|
||||
"os"
|
||||
"time"
|
||||
|
||||
"github.com/mrleo1nid/pzmanager/internal/config"
|
||||
"github.com/mrleo1nid/pzmanager/internal/pzconfig"
|
||||
"github.com/mrleo1nid/pzmanager/internal/pzserver"
|
||||
)
|
||||
|
||||
// validateProfileMemory проверяет границы кучи JVM здесь, а не при запуске
|
||||
// сервера: иначе про опечатку в профиле стало бы известно только из лога
|
||||
// упавшего сервера.
|
||||
func validateProfileMemory(dto profileDTO) error {
|
||||
// validateProfile проверяет числа из формы здесь, а не при запуске сервера:
|
||||
// иначе про опечатку в профиле стало бы известно только из лога упавшего
|
||||
// сервера.
|
||||
func validateProfile(dto profileDTO) error {
|
||||
maxBytes, err := pzconfig.ParseMemory(dto.JavaMemory)
|
||||
if err != nil {
|
||||
return err
|
||||
@@ -27,9 +28,27 @@ func validateProfileMemory(dto profileDTO) error {
|
||||
return fmt.Errorf("начальный размер памяти (%s) больше максимального (%s)",
|
||||
dto.JavaMemoryMin, dto.JavaMemory)
|
||||
}
|
||||
if dto.StartupTimeoutMin < 0 || dto.StartupTimeoutMin > maxStartupTimeoutMin {
|
||||
return fmt.Errorf("ожидание готовности должно быть от 1 до %d минут",
|
||||
maxStartupTimeoutMin)
|
||||
}
|
||||
return nil
|
||||
}
|
||||
|
||||
// startupTimeout переводит минуты из формы в длительность. Ноль оставляем
|
||||
// нулём: он означает «как по умолчанию», и записывать в профиль конкретное
|
||||
// число значило бы заморозить его при смене умолчания.
|
||||
func startupTimeout(dto profileDTO) config.Duration {
|
||||
if dto.StartupTimeoutMin <= 0 {
|
||||
return 0
|
||||
}
|
||||
return config.Duration(time.Duration(dto.StartupTimeoutMin) * time.Minute)
|
||||
}
|
||||
|
||||
// maxStartupTimeoutMin — потолок на ожидание готовности. Больше трёх часов
|
||||
// это уже не «долго грузится», а зависший сервер.
|
||||
const maxStartupTimeoutMin = 180
|
||||
|
||||
// profileDTO — профиль в том виде, в каком его показывает панель. Пароль
|
||||
// наружу не отдаётся, только признак того, что он задан.
|
||||
type profileDTO struct {
|
||||
@@ -39,7 +58,10 @@ type profileDTO struct {
|
||||
JavaMemoryMin string `json:"java_memory_min"`
|
||||
ExtraArgs []string `json:"extra_args"`
|
||||
HasAdminPassword bool `json:"has_admin_password"`
|
||||
AdminPassword string `json:"admin_password,omitempty"`
|
||||
// StartupTimeoutMin — сколько минут ждать строку о готовности сервера.
|
||||
// Ноль означает значение по умолчанию, его панель показывает подсказкой.
|
||||
StartupTimeoutMin int `json:"startup_timeout_min"`
|
||||
AdminPassword string `json:"admin_password,omitempty"`
|
||||
// Признаки состояния на диске: по ним панель подсказывает, что профиль
|
||||
// ещё ни разу не запускался.
|
||||
HasConfig bool `json:"has_config"`
|
||||
@@ -51,15 +73,16 @@ type profileDTO struct {
|
||||
|
||||
func (s *Server) profileToDTO(p config.Profile) profileDTO {
|
||||
return profileDTO{
|
||||
ID: p.ID,
|
||||
Title: p.Title,
|
||||
JavaMemory: p.JavaMemory,
|
||||
JavaMemoryMin: p.JavaMemoryMin,
|
||||
ExtraArgs: p.ExtraArgs,
|
||||
HasAdminPassword: p.AdminPassword != "",
|
||||
HasConfig: fileExists(s.cfg.ServerConfigPathFor(p.ID)),
|
||||
WorldReady: fileExists(s.cfg.WorldDirFor(p.ID)),
|
||||
Active: p.ID == s.cfg.Active,
|
||||
ID: p.ID,
|
||||
Title: p.Title,
|
||||
JavaMemory: p.JavaMemory,
|
||||
JavaMemoryMin: p.JavaMemoryMin,
|
||||
ExtraArgs: p.ExtraArgs,
|
||||
HasAdminPassword: p.AdminPassword != "",
|
||||
StartupTimeoutMin: int(p.StartupTimeout.Duration().Minutes()),
|
||||
HasConfig: fileExists(s.cfg.ServerConfigPathFor(p.ID)),
|
||||
WorldReady: fileExists(s.cfg.WorldDirFor(p.ID)),
|
||||
Active: p.ID == s.cfg.Active,
|
||||
}
|
||||
}
|
||||
|
||||
@@ -69,8 +92,9 @@ func (s *Server) handleListProfiles(w http.ResponseWriter, r *http.Request) {
|
||||
profiles = append(profiles, s.profileToDTO(p))
|
||||
}
|
||||
writeJSON(w, http.StatusOK, map[string]any{
|
||||
"profiles": profiles,
|
||||
"active": s.cfg.Active,
|
||||
"profiles": profiles,
|
||||
"active": s.cfg.Active,
|
||||
"default_startup_timeout_min": int(config.DefaultStartupTimeout.Minutes()),
|
||||
// Переключать профиль можно только на остановленном сервере.
|
||||
"can_switch": s.sup.State() == pzserver.StateStopped,
|
||||
})
|
||||
@@ -83,18 +107,19 @@ func (s *Server) handleCreateProfile(w http.ResponseWriter, r *http.Request) {
|
||||
if !decodeJSON(w, r, &dto) {
|
||||
return
|
||||
}
|
||||
if err := validateProfileMemory(dto); err != nil {
|
||||
if err := validateProfile(dto); err != nil {
|
||||
writeError(w, r, http.StatusBadRequest, err.Error())
|
||||
return
|
||||
}
|
||||
|
||||
profile := config.Profile{
|
||||
ID: dto.ID,
|
||||
Title: dto.Title,
|
||||
JavaMemory: dto.JavaMemory,
|
||||
JavaMemoryMin: dto.JavaMemoryMin,
|
||||
ExtraArgs: dto.ExtraArgs,
|
||||
AdminPassword: dto.AdminPassword,
|
||||
ID: dto.ID,
|
||||
Title: dto.Title,
|
||||
JavaMemory: dto.JavaMemory,
|
||||
JavaMemoryMin: dto.JavaMemoryMin,
|
||||
ExtraArgs: dto.ExtraArgs,
|
||||
AdminPassword: dto.AdminPassword,
|
||||
StartupTimeout: startupTimeout(dto),
|
||||
}
|
||||
if err := s.cfg.AddProfile(profile); err != nil {
|
||||
writeError(w, r, http.StatusBadRequest, err.Error())
|
||||
@@ -160,17 +185,18 @@ func (s *Server) handleUpdateProfile(w http.ResponseWriter, r *http.Request) {
|
||||
if !decodeJSON(w, r, &dto) {
|
||||
return
|
||||
}
|
||||
if err := validateProfileMemory(dto); err != nil {
|
||||
if err := validateProfile(dto); err != nil {
|
||||
writeError(w, r, http.StatusBadRequest, err.Error())
|
||||
return
|
||||
}
|
||||
|
||||
updated := config.Profile{
|
||||
ID: id,
|
||||
Title: dto.Title,
|
||||
JavaMemory: dto.JavaMemory,
|
||||
JavaMemoryMin: dto.JavaMemoryMin,
|
||||
ExtraArgs: dto.ExtraArgs,
|
||||
ID: id,
|
||||
Title: dto.Title,
|
||||
JavaMemory: dto.JavaMemory,
|
||||
JavaMemoryMin: dto.JavaMemoryMin,
|
||||
ExtraArgs: dto.ExtraArgs,
|
||||
StartupTimeout: startupTimeout(dto),
|
||||
// Пустой пароль означает "оставить прежний": наружу мы его не отдаём,
|
||||
// и форма присылает пустую строку, пока его не меняли.
|
||||
AdminPassword: current.AdminPassword,
|
||||
|
||||
Reference in New Issue
Block a user