local DNS uses mode tcp_and_udp by default

- Fixes #1281
shadowsocks · Aug 31, 2023 · 6464f37 · 6464f37
1 parent fbe81b7
commit 6464f37
Showing 1 changed file with 19 additions and 10 deletions.
diff --git a/crates/shadowsocks-service/src/config.rs b/crates/shadowsocks-service/src/config.rs
@@ -66,14 +66,7 @@ use serde::{Deserialize, Serialize};
 use shadowsocks::relay::socks5::Address;
 use shadowsocks::{
     config::{
-        ManagerAddr,
-        Mode,
-        ReplayAttackPolicy,
-        ServerAddr,
-        ServerConfig,
-        ServerUser,
-        ServerUserManager,
-        ServerWeight,
+        ManagerAddr, Mode, ReplayAttackPolicy, ServerAddr, ServerConfig, ServerUser, ServerUserManager, ServerWeight,
     },
     crypto::CipherKind,
     plugin::PluginConfig,
@@ -891,12 +884,20 @@ pub struct LocalConfig {
 impl LocalConfig {
     /// Create a new `LocalConfig`
     pub fn new(protocol: ProtocolType) -> LocalConfig {
+        // DNS server runs in `TcpAndUdp` mode by default to maintain backwards compatibility
+        // see https://github.com/shadowsocks/shadowsocks-rust/issues/1281
+        let mode = match protocol {
+            #[cfg(feature = "local-dns")]
+            ProtocolType::Dns => Mode::TcpAndUdp,
+            _ => Mode::TcpOnly,
+        };
+
         LocalConfig {
             addr: None,
 
             protocol,
 
-            mode: Mode::TcpOnly,
+            mode,
             udp_addr: None,
 
             #[cfg(feature = "local-tunnel")]
@@ -1456,7 +1457,15 @@ impl Config {
                                 }
                             },
                             None => {
-                                local_config.mode = global_mode;
+                                // DNS server runs in `TcpAndUdp` mode by default to maintain backwards compatibility
+                                // see https://github.com/shadowsocks/shadowsocks-rust/issues/1281
+                                let mode = match protocol {
+                                    #[cfg(feature = "local-dns")]
+                                    ProtocolType::Dns => Mode::TcpAndUdp,
+                                    _ => global_mode,
+                                };
+
+                                local_config.mode = mode;
                             }
                         }