perf: Improve config validation error messages and handling

This commit is contained in:
wonfen 2025-02-25 13:47:29 +08:00
parent bc5d577553
commit 7a0e38a1b4
11 changed files with 202 additions and 195 deletions

View File

@ -13,7 +13,6 @@ use sysproxy::{Autoproxy, Sysproxy};
type CmdResult<T = ()> = Result<T, String>;
use reqwest_dav::list_cmd::ListFile;
use tauri::Manager;
use tauri_plugin_shell::ShellExt;
use std::fs;
#[tauri::command]
@ -30,10 +29,24 @@ pub fn get_profiles() -> CmdResult<IProfiles> {
#[tauri::command]
pub async fn enhance_profiles() -> CmdResult {
wrap_err!(CoreManager::global().update_config().await)?;
match CoreManager::global().update_config().await {
Ok((true, _)) => {
println!("[enhance_profiles] 配置更新成功");
log_err!(tray::Tray::global().update_tooltip());
handle::Handle::refresh_clash();
Ok(())
}
Ok((false, error_msg)) => {
println!("[enhance_profiles] 配置验证失败: {}", error_msg);
handle::Handle::notice_message("config_validate::error", &error_msg);
Ok(())
}
Err(e) => {
println!("[enhance_profiles] 更新过程发生错误: {}", e);
handle::Handle::notice_message("config_validate::process_terminated", &e.to_string());
Ok(())
}
}
}
#[tauri::command]
@ -72,7 +85,7 @@ pub async fn delete_profile(index: String) -> CmdResult {
#[tauri::command]
pub async fn patch_profiles_config(
profiles: IProfiles
) -> CmdResult {
) -> CmdResult<bool> {
println!("[cmd配置patch] 开始修改配置文件");
// 保存当前配置,以便在验证失败时恢复
@ -85,21 +98,17 @@ pub async fn patch_profiles_config(
// 更新配置并进行验证
match CoreManager::global().update_config().await {
Ok(_) => {
Ok((true, _)) => {
println!("[cmd配置patch] 配置更新成功");
handle::Handle::refresh_clash();
let _ = tray::Tray::global().update_tooltip();
Config::profiles().apply();
wrap_err!(Config::profiles().data().save_file())?;
// 发送成功通知
handle::Handle::notice_message("operation_success", "配置patch成功");
Ok(())
Ok(true)
}
Err(err) => {
println!("[cmd配置patch] 更新配置失败: {}", err);
Ok((false, error_msg)) => {
println!("[cmd配置patch] 配置验证失败: {}", error_msg);
Config::profiles().discard();
println!("[cmd配置patch] 错误详情: {}", err);
// 如果验证失败,恢复到之前的配置
if let Some(prev_profile) = current_profile {
@ -115,7 +124,15 @@ pub async fn patch_profiles_config(
println!("[cmd配置patch] 成功恢复到之前的配置");
}
Err(err.to_string())
// 发送验证错误通知
handle::Handle::notice_message("config_validate::error", &error_msg);
Ok(false)
}
Err(e) => {
println!("[cmd配置patch] 更新过程发生错误: {}", e);
Config::profiles().discard();
handle::Handle::notice_message("config_validate::boot_error", &e.to_string());
Ok(false)
}
}
}
@ -125,7 +142,7 @@ pub async fn patch_profiles_config(
pub async fn patch_profiles_config_by_profile_index(
_app_handle: tauri::AppHandle,
profile_index: String
) -> CmdResult {
) -> CmdResult<bool> {
let profiles = IProfiles{current: Some(profile_index), items: None};
patch_profiles_config(profiles).await
}
@ -183,39 +200,29 @@ pub async fn save_profile_file(index: String, file_data: Option<String>) -> CmdR
// 保存新的配置文件
wrap_err!(fs::write(&file_path, file_data.clone().unwrap()))?;
// 直接验证保存的配置文件
let clash_core = Config::verge().latest().clash_core.clone().unwrap_or("verge-mihomo".into());
let test_dir = wrap_err!(dirs::app_home_dir())?.join("test");
let test_dir = test_dir.to_string_lossy();
let file_path_str = file_path.to_string_lossy();
println!("[cmd配置save] 开始验证配置文件: {}", file_path_str);
let app_handle = handle::Handle::global().app_handle().unwrap();
let output = wrap_err!(app_handle
.shell()
.sidecar(clash_core)
.map_err(|e| e.to_string())?
.args(["-t", "-d", test_dir.as_ref(), "-f", file_path_str.as_ref()])
.output()
.await)?;
let stderr = String::from_utf8_lossy(&output.stderr);
let error_keywords = ["FATA", "fatal", "Parse config error", "level=fatal"];
let has_error = !output.status.success() || error_keywords.iter().any(|&kw| stderr.contains(kw));
if has_error {
println!("[cmd配置save] 编辑验证失败 {}", stderr);
// 验证配置文件
match CoreManager::global().validate_config_file(&file_path_str).await {
Ok((true, _)) => {
println!("[cmd配置save] 验证成功");
Ok(())
}
Ok((false, error_msg)) => {
println!("[cmd配置save] 验证失败: {}", error_msg);
// 恢复原始配置文件
wrap_err!(fs::write(&file_path, original_content))?;
// 发送错误通知
handle::Handle::notice_message("config_validate::error", &*stderr);
return Err(format!("cmd配置save失败 {}", stderr));
}
println!("[cmd配置save] 验证成功");
handle::Handle::notice_message("operation_success", "配置更新成功");
handle::Handle::notice_message("config_validate::error", &error_msg.to_string()); // 保存文件弹出此提示
Ok(())
}
Err(e) => {
println!("[cmd配置save] 验证过程发生错误: {}", e);
// 恢复原始配置文件
wrap_err!(fs::write(&file_path, original_content))?;
Err(e.to_string())
}
}
}
#[tauri::command]

View File

@ -80,24 +80,11 @@ impl Config {
match CoreManager::global().validate_config().await {
Ok((is_valid, error_msg)) => {
if !is_valid {
println!("[首次启动] 配置验证失败,使用默认最小配置启动{}", error_msg);
if error_msg.is_empty() {
println!("[首次启动] 配置验证失败,使用默认最小配置启动: {}", error_msg);
CoreManager::global()
.use_default_config(
"config_validate::boot_error",
"",
)
.use_default_config("config_validate::boot_error", &error_msg)
.await?;
Some(("config_validate::boot_error", String::new()))
} else {
CoreManager::global()
.use_default_config(
"config_validate::stderr_error",
&error_msg,
)
.await?;
Some(("config_validate::stderr_error", error_msg))
}
Some(("config_validate::boot_error", error_msg))
} else {
println!("[首次启动] 配置验证成功");
Some(("config_validate::success", String::new()))
@ -106,10 +93,7 @@ impl Config {
Err(err) => {
println!("[首次启动] 验证进程执行失败: {}", err);
CoreManager::global()
.use_default_config(
"config_validate::process_terminated",
"",
)
.use_default_config("config_validate::process_terminated", "")
.await?;
Some(("config_validate::process_terminated", String::new()))
}

View File

@ -117,62 +117,63 @@ impl CoreManager {
}
log::info!(target: "app", "change core to `{clash_core}`");
Config::verge().draft().clash_core = Some(clash_core);
Config::generate().await?;
// 验证配置
println!("[切换内核] 开始验证配置");
// 1. 先更新内核配置(但不应用)
Config::verge().draft().clash_core = Some(clash_core);
// 2. 使用新内核验证配置
println!("[切换内核] 使用新内核验证配置");
match self.validate_config().await {
Ok((is_valid, error_msg)) => {
if !is_valid {
println!("[切换内核] 配置验证失败: {}", error_msg);
if error_msg.is_empty() {
self.use_default_config(
"config_validate::core_change",
"",
).await?;
} else {
self.use_default_config(
"config_validate::stderr_error",
&error_msg,
).await?;
}
} else {
println!("[切换内核] 配置验证成功");
handle::Handle::notice_message("config_validate::success", "");
}
}
Err(err) => {
println!("[切换内核] 验证进程执行失败: {}", err);
self.use_default_config(
"config_validate::process_terminated",
"",
).await?;
}
}
Ok((true, _)) => {
println!("[切换内核] 配置验证通过,开始切换内核");
// 3. 验证通过后,应用内核配置并重启
Config::verge().apply();
log_err!(Config::verge().latest().save_file());
match self.restart_core().await {
Ok(_) => {
Config::verge().apply();
println!("[切换内核] 内核切换成功");
Config::runtime().apply();
log_err!(Config::verge().latest().save_file());
Ok(())
}
Err(err) => {
println!("[切换内核] 内核切换失败: {}", err);
Config::verge().discard();
Config::runtime().discard();
Err(err)
}
}
}
Ok((false, error_msg)) => {
println!("[切换内核] 配置验证失败: {}", error_msg);
// 使用默认配置并继续切换内核
self.use_default_config("config_validate::core_change", &error_msg).await?;
Config::verge().apply();
log_err!(Config::verge().latest().save_file());
/// 使用子进程验证配置
pub async fn validate_config(&self) -> Result<(bool, String)> {
println!("[core配置验证] 开始验证配置");
match self.restart_core().await {
Ok(_) => {
println!("[切换内核] 内核切换成功(使用默认配置)");
Ok(())
}
Err(err) => {
println!("[切换内核] 内核切换失败: {}", err);
Config::verge().discard();
Err(err)
}
}
}
Err(err) => {
println!("[切换内核] 验证过程发生错误: {}", err);
Config::verge().discard();
Err(err)
}
}
}
let config_path = Config::generate_file(ConfigType::Check)?;
let config_path = dirs::path_to_str(&config_path)?;
println!("[core配置验证] 配置文件路径: {}", config_path);
/// 内部验证配置文件的实现
async fn validate_config_internal(&self, config_path: &str) -> Result<(bool, String)> {
println!("[core配置验证] 开始验证配置文件: {}", config_path);
let clash_core = { Config::verge().latest().clash_core.clone() };
let clash_core = clash_core.unwrap_or("verge-mihomo".into());
@ -199,36 +200,51 @@ impl CoreManager {
let error_keywords = ["FATA", "fatal", "Parse config error", "level=fatal"];
let has_error = !output.status.success() || error_keywords.iter().any(|&kw| stderr.contains(kw));
println!("[core配置验证] 退出状态: {:?}", output.status);
println!("\n[core配置验证] -------- 验证结果 --------");
println!("[core配置验证] 进程退出状态: {:?}", output.status);
if !stderr.is_empty() {
println!("[core配置验证] 错误输出: {}", stderr);
println!("[core配置验证] stderr输出:\n{}", stderr);
}
if !stdout.is_empty() {
println!("[core配置验证] 标准输出: {}", stdout);
println!("[core配置验证] stdout输出:\n{}", stdout);
}
if has_error {
let error_msg = if stderr.is_empty() {
if let Some(code) = output.status.code() {
handle::Handle::notice_message("config_validate::error", &code.to_string());
String::new()
println!("[core配置验证] 发现错误,开始处理错误信息");
let error_msg = if !stdout.is_empty() {
stdout.to_string()
} else if !stderr.is_empty() {
stderr.to_string()
} else if let Some(code) = output.status.code() {
format!("验证进程异常退出,退出码: {}", code)
} else {
handle::Handle::notice_message("config_validate::process_terminated", "");
String::new()
}
} else {
handle::Handle::notice_message("config_validate::stderr_error", &*stderr);
String::new()
"验证进程被终止".to_string()
};
Ok((false, error_msg))
println!("[core配置验证] -------- 验证结束 --------\n");
Ok((false, error_msg)) // 返回错误消息给调用者处理
} else {
handle::Handle::notice_message("config_validate::success", "");
println!("[core配置验证] 验证成功");
println!("[core配置验证] -------- 验证结束 --------\n");
Ok((true, String::new()))
}
}
/// 验证运行时配置
pub async fn validate_config(&self) -> Result<(bool, String)> {
let config_path = Config::generate_file(ConfigType::Check)?;
let config_path = dirs::path_to_str(&config_path)?;
self.validate_config_internal(config_path).await
}
/// 验证指定的配置文件
pub async fn validate_config_file(&self, config_path: &str) -> Result<(bool, String)> {
self.validate_config_internal(config_path).await
}
/// 更新proxies等配置
pub async fn update_config(&self) -> Result<()> {
pub async fn update_config(&self) -> Result<(bool, String)> {
println!("[core配置更新] 开始更新配置");
// 1. 先生成新的配置内容
@ -242,48 +258,48 @@ impl CoreManager {
println!("[core配置更新] 临时配置文件路径: {}", temp_config);
// 3. 验证配置
let (is_valid, error_msg) = match self.validate_config().await {
Ok((valid, msg)) => (valid, msg),
Err(e) => {
println!("[core配置更新] 验证过程发生错误: {}", e);
Config::runtime().discard(); // 验证失败时丢弃新配置
return Err(e);
}
};
if !is_valid {
println!("[core配置更新] 配置验证未通过,保持当前配置不变");
Config::runtime().discard(); // 验证失败时丢弃新配置
return Err(anyhow::anyhow!(error_msg));
}
match self.validate_config().await {
Ok((true, _)) => {
println!("[core配置更新] 配置验证通过");
// 4. 验证通过后,生成正式的运行时配置
println!("[core配置更新] 验证通过,生成运行时配置");
println!("[core配置更新] 生成运行时配置");
let run_path = Config::generate_file(ConfigType::Run)?;
let run_path = dirs::path_to_str(&run_path)?;
// 5. 应用新配置
println!("[core配置更新] 应用新配置");
for i in 0..10 {
for i in 0..3 {
match clash_api::put_configs(run_path).await {
Ok(_) => {
println!("[core配置更新] 配置应用成功");
Config::runtime().apply(); // 应用成功时保存新配置
break;
Config::runtime().apply();
return Ok((true, String::new()));
}
Err(err) => {
if i < 9 {
if i < 2 {
println!("[core配置更新] 第{}次重试应用配置", i + 1);
log::info!(target: "app", "{err}");
sleep(Duration::from_millis(100)).await;
} else {
println!("[core配置更新] 配置应用失败: {}", err);
Config::runtime().discard(); // 应用失败时丢弃新配置
return Err(err.into());
Config::runtime().discard();
return Ok((false, err.to_string()));
}
}
}
sleep(Duration::from_millis(100)).await;
}
Ok(())
Ok((true, String::new()))
}
Ok((false, error_msg)) => {
println!("[core配置更新] 配置验证失败: {}", error_msg);
Config::runtime().discard();
Ok((false, error_msg))
}
Err(e) => {
println!("[core配置更新] 验证过程发生错误: {}", e);
Config::runtime().discard();
Err(e)
}
}
}
}

View File

@ -439,8 +439,8 @@
"Enable Tray Speed": "تفعيل سرعة التراي",
"Lite Mode": "وضع الأداء الخفيف",
"Lite Mode Info": "إيقاف الواجهة الرسومية والإبقاء على تشغيل النواة",
"Config Validation Failed": "فشل التحقق من تكوين الاشتراك. يرجى التحقق من ملف تكوين الاشتراك؛ تم التراجع عن التعديلات.",
"Boot Config Validation Failed": "فشل التحقق من تكوين الاشتراك. تم البدء بالتكوين الافتراضي. يرجى التحقق من ملف تكوين الاشتراك.",
"Core Change Config Validation Failed": "فشل التحقق من التكوين عند تغيير النواة؛ تم البدء بالتكوين الافتراضي. يرجى التحقق من ملف تكوين الاشتراك.",
"Config Validation Process Terminated": "تم إنهاء عملية التحقق من التكوين."
"Config Validation Failed": "فشل التحقق من تكوين الاشتراك، يرجى فحص ملف التكوين، تم التراجع عن التغييرات، تفاصيل الخطأ:",
"Boot Config Validation Failed": "فشل التحقق من التكوين عند الإقلاع، تم استخدام التكوين الافتراضي، يرجى فحص ملف التكوين، تفاصيل الخطأ:",
"Core Change Config Validation Failed": "فشل التحقق من التكوين عند تغيير النواة، تم استخدام التكوين الافتراضي، يرجى فحص ملف التكوين، تفاصيل الخطأ:",
"Config Validation Process Terminated": "تم إنهاء عملية التحقق"
}

View File

@ -436,8 +436,8 @@
"Enable Tray Speed": "فعال کردن سرعت ترای",
"Lite Mode": "در فارسی",
"Lite Mode Info": "رابط کاربری گرافیکی را ببندید و فقط هسته را در حال اجرا نگه دارید",
"Config Validation Failed": "اعتبارسنجی پیکربندی اشتراک ناموفق بود. لطفاً فایل پیکربندی اشتراک را بررسی کنید؛ تغییرات به حالت قبل بازگردانده شد.",
"Boot Config Validation Failed": "اعتبارسنجی پیکربندی اشتراک ناموفق بود. با پیکربندی پیش‌فرض راه‌اندازی شد. لطفاً فایل پیکربندی اشتراک را بررسی کنید.",
"Core Change Config Validation Failed": "اعتبارسنجی پیکربندی هنگام تغییر هسته ناموفق بود؛ با پیکربندی پیش‌فرض راه‌اندازی شد. لطفاً فایل پیکربندی اشتراک را بررسی کنید.",
"Config Validation Process Terminated": "فرآیند اعتبارسنجی پیکربندی متوقف شد."
"Config Validation Failed": "اعتبارسنجی پیکربندی اشتراک ناموفق بود، فایل پیکربندی را بررسی کنید، تغییرات برگشت داده شد، جزئیات خطا:",
"Boot Config Validation Failed": "اعتبارسنجی پیکربندی هنگام راه‌اندازی ناموفق بود، پیکربندی پیش‌فرض استفاده شد، فایل پیکربندی را بررسی کنید، جزئیات خطا:",
"Core Change Config Validation Failed": "اعتبارسنجی پیکربندی هنگام تغییر هسته ناموفق بود، پیکربندی پیش‌فرض استفاده شد، فایل پیکربندی را بررسی کنید، جزئیات خطا:",
"Config Validation Process Terminated": "فرآیند اعتبارسنجی متوقف شد"
}

View File

@ -435,8 +435,8 @@
"Enable Tray Speed": "Aktifkan Tray Speed",
"Lite Mode": "Mode Ringan",
"Lite Mode Info": "Tutup GUI dan biarkan hanya kernel yang berjalan",
"Config Validation Failed": "Validasi konfigurasi langganan gagal. Silakan periksa file konfigurasi langganan; perubahan telah dikembalikan.",
"Boot Config Validation Failed": "Validasi konfigurasi langganan gagal. Memulai dengan konfigurasi default. Silakan periksa file konfigurasi langganan.",
"Core Change Config Validation Failed": "Validasi konfigurasi gagal saat beralih kernel; memulai dengan konfigurasi default. Silakan periksa file konfigurasi langganan.",
"Config Validation Process Terminated": "Proses validasi konfigurasi telah dihentikan."
"Config Validation Failed": "Validasi konfigurasi langganan gagal, periksa file konfigurasi, perubahan dibatalkan, detail kesalahan:",
"Boot Config Validation Failed": "Validasi konfigurasi saat boot gagal, menggunakan konfigurasi default, periksa file konfigurasi, detail kesalahan:",
"Core Change Config Validation Failed": "Validasi konfigurasi saat ganti inti gagal, menggunakan konfigurasi default, periksa file konfigurasi, detail kesalahan:",
"Config Validation Process Terminated": "Proses validasi dihentikan"
}

View File

@ -436,8 +436,8 @@
"Enable Tray Speed": "Включить скорость в лотке",
"Lite Mode": "Облегченный режим",
"Lite Mode Info": "Закройте графический интерфейс и оставьте работать только ядро",
"Config Validation Failed": "Проверка конфигурации подписки не удалась. Пожалуйста, проверьте файл конфигурации подписки; изменения были отменены.",
"Boot Config Validation Failed": "Проверка конфигурации подписки не удалась. Запуск с конфигурацией по умолчанию. Пожалуйста, проверьте файл конфигурации подписки.",
"Core Change Config Validation Failed": "Проверка конфигурации при смене ядра не удалась; запуск с конфигурацией по умолчанию. Пожалуйста, проверьте файл конфигурации подписки.",
"Config Validation Process Terminated": "Процесс проверки конфигурации был завершён."
"Config Validation Failed": "Ошибка проверки конфигурации подписки, проверьте файл конфигурации, изменения отменены, ошибка:",
"Boot Config Validation Failed": "Ошибка проверки конфигурации при запуске, используется конфигурация по умолчанию, проверьте файл конфигурации, ошибка:",
"Core Change Config Validation Failed": "Ошибка проверки конфигурации при смене ядра, используется конфигурация по умолчанию, проверьте файл конфигурации, ошибка:",
"Config Validation Process Terminated": "Процесс проверки прерван"
}

View File

@ -435,8 +435,8 @@
"Enable Tray Speed": "Трей скоростьне үстерү",
"Lite Mode": "Җиңел Режим",
"Lite Mode Info": "GUI-ны ябыгыз һәм бары тик төшне генә эшләтеп калдырыгыз",
"Config Validation Failed": "Язылу көйләүләрен тикшерү уңышсыз тәмамланды. Зинһар, язылу көйләүләре файлын тикшерегез; үзгәрешләр кире кагылды.",
"Boot Config Validation Failed": "Язылу көйләүләрен тикшерү уңышсыз тәмамланды. Кәгазь көйләүләр белән эшли башлады. Зинһар, язылу көйләүләре файлын тикшерегез.",
"Core Change Config Validation Failed": "Ядроны алыштырганда көйләүне тикшерү уңышсыз тәмамланды; кәгазь көйләүләр белән эшли башлады. Зинһар, язылу көйләүләре файлын тикшерегез.",
"Config Validation Process Terminated": "Көйләүне тикшерү процессы туктатылды."
"Config Validation Failed": "Язылу көйләү тикшерүе уңышсыз, көйләү файлын тикшерегез, үзгәрешләр кире кайтарылды, хата:",
"Boot Config Validation Failed": "Йөкләү вакытында көйләү тикшерүе уңышсыз, стандарт көйләү кулланылды, көйләү файлын тикшерегез, хата:",
"Core Change Config Validation Failed": "Ядро алыштырганда көйләү тикшерүе уңышсыз, стандарт көйләү кулланылды, көйләү файлын тикшерегез, хата:",
"Config Validation Process Terminated": "Тикшерү процессы туктатылды"
}

View File

@ -436,8 +436,8 @@
"Enable Tray Speed": "启用托盘速率",
"Lite Mode": "轻量模式",
"Lite Mode Info": "关闭GUI界面仅保留内核运行",
"Config Validation Failed": "订阅配置校验失败,请检查订阅配置文件,修改已回滚",
"Boot Config Validation Failed": "启动订阅配置校验失败,已使用默认配置启动;请检查订阅配置文件",
"Core Change Config Validation Failed": "切换内核时配置校验失败,已使用默认配置启动;请检查订阅配置文件",
"Config Validation Failed": "订阅配置校验失败,请检查订阅配置文件,变更已回滚,错误详情:",
"Boot Config Validation Failed": "启动订阅配置校验失败,已使用默认配置启动;请检查订阅配置文件,错误详情:",
"Core Change Config Validation Failed": "切换内核时配置校验失败,已使用默认配置启动;请检查订阅配置文件,错误详情:",
"Config Validation Process Terminated": "验证进程被终止"
}

View File

@ -58,19 +58,19 @@ const handleNoticeMessage = (
Notice.error(msg);
break;
case "config_validate::boot_error":
Notice.error(t("Boot Config Validation Failed"));
Notice.error(`${t("Boot Config Validation Failed")} ${msg}`);
break;
case "config_validate::core_change":
Notice.error(t("Core Change Config Validation Failed"));
Notice.error(`${t("Core Change Config Validation Failed")} ${msg}`);
break;
case "config_validate::error":
Notice.error(t("Config Validation Failed"));
Notice.error(`${t("Config Validation Failed")} ${msg}`);
break;
case "config_validate::process_terminated":
Notice.error(t("Config Validation Process Terminated"));
break;
case "config_validate::stderr_error":
Notice.error(msg);
case "config_validate::stdout_error":
Notice.error(`${t("Config Validation Failed")} ${msg}`);
break;
}
};

View File

@ -174,11 +174,11 @@ const ProfilePage = () => {
}, 100);
try {
await patchProfiles({ current: profile });
const success = await patchProfiles({ current: profile });
await mutateLogs();
closeAllConnections();
await activateSelected();
if (notifySuccess) {
if (notifySuccess && success) {
Notice.success(t("Profile Switched"), 1000);
}
} catch (err: any) {