fix(dns): DNS failover now applies changes to OS + Linux NM integration#1
Conversation
- dns.rs: +292 líneas con soporte Linux (nmcli, resolvectl, NetworkManager) - dns_intelligence.rs: +50 líneas para apply real del failover al SO (antes solo actualizaba memoria, nunca llamaba a set_dns()) - Add Tauri gen/schemas y capabilities loust <davidmirelesll@outlook.com>
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (5)
📒 Files selected for processing (26)
📝 WalkthroughWalkthroughThis PR migrates the Tauri desktop application from v1 to v2 APIs (dependencies, configuration, capability file, tray/notification plugins) and simultaneously adds Linux-native implementations across all backend Rust commands and services (DNS via NetworkManager/nmcli, diagnostics via ip/ping/nslookup, optimizer via sysctl, monitoring via /sys/class/net, updater via AppImage/xdg-open) and updates all Svelte frontend components to detect the runtime platform and render platform-appropriate labels, handlers, and tool actions. ChangesTauri v2 + Linux Cross-Platform Migration
Estimated code review effort🎯 5 (Critical) | ⏱️ ~120 minutes Poem
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
DNS failover was detecting failures and updating internal state, but never actually calling
set_dns()to apply the change to the operating system. This caused DNS probe errors to persist even when the app detected the issue.Changes
dns_intelligence.rs(+50 lines):check_failover_static()now callsset_dns()to apply the failover to the OS after detectingconsecutive_failures >= max_consecutive_failures.dns.rs(+292 lines): Full Linux NetworkManager integration —nmcli-based adapter detection, connection reapply,resolvectlsupport, DNS provider list, latency measurement, and DNS resolution checks.capabilities/migrated.json: Narrowed PowerShell validator from\\S+(any string) to specific DNS cmdlets only.Scope
Testing
cargo check # ✅ compilesSecurity
PowerShell capability narrowed to:
Set-DnsClientServerAddress,Clear-DnsClientCache,Test-Connection,Get-NetAdapter,Get-DnsClientServerAddress,Resolve-DnsNameSummary by CodeRabbit
New Features
Bug Fixes
Chores