fix: deduplicate routes when multiple domains resolve to same IP

Author: GeiserX

Casks/vpn-bypass.rb

@@ -3,7 +3,7 @@
 # Or if using local tap: brew install --cask --no-quarantine ./Casks/vpn-bypass.rb
 
 cask "vpn-bypass" do
-  version "1.6.0"
+  version "1.6.1"
   sha256 "37b127a55aec0bdb80e824e59e840ce5b529c09086aac7fc24dc4616abb817bd"
 
   url "https://github.com/GeiserX/VPNBypass/releases/download/v#{version}/VPNBypass-#{version}.dmg"

Info.plist

@@ -17,9 +17,9 @@
 	<key>CFBundlePackageType</key>
 	<string>APPL</string>
 	<key>CFBundleShortVersionString</key>
-	<string>1.6.0</string>
+	<string>1.6.1</string>
 	<key>CFBundleVersion</key>
-	<string>9</string>
+	<string>10</string>
 	<key>LSMinimumSystemVersion</key>
 	<string>13.0</string>
 	<key>LSUIElement</key>

README.md

@@ -11,7 +11,7 @@
 <p align="center">
   <img src="https://img.shields.io/badge/macOS-13%2B-blue" alt="macOS 13+">
   <img src="https://img.shields.io/badge/Swift-5.9-orange" alt="Swift 5.9">
-  <a href="https://github.com/GeiserX/vpn-macos-bypass/releases"><img src="https://img.shields.io/badge/version-1.6.0-green" alt="Version"></a>
+  <a href="https://github.com/GeiserX/vpn-macos-bypass/releases"><img src="https://img.shields.io/badge/version-1.6.1-green" alt="Version"></a>
 </p>
 
 ## Why?

Sources/RouteManager.swift

@@ -934,6 +934,7 @@ final class RouteManager: ObservableObject {
 
         // Collect all routes to add (for batch operation)
         var routesToAdd: [(destination: String, gateway: String, isNetwork: Bool, source: String)] = []
+        var seenDestinations: Set<String> = []  // Deduplicate by destination IP
 
         while index < allDomains.count {
             let endIndex = min(index + batchSize, allDomains.count)
@@ -964,7 +965,8 @@ final class RouteManager: ObservableObject {
                     }
                     dnsDiskCache[result.domain] = ips  // Update persistent cache
 
-                    for ip in ips {
+                    for ip in ips where !seenDestinations.contains(ip) {
+                        seenDestinations.insert(ip)
                         routesToAdd.append((destination: ip, gateway: gateway, isNetwork: false, source: result.source))
                     }
                 } else if let cachedIPs = dnsDiskCache[result.domain], !cachedIPs.isEmpty {
@@ -973,7 +975,8 @@ final class RouteManager: ObservableObject {
                     if let firstIP = cachedIPs.first {
                         dnsCache[result.domain] = firstIP
                     }
-                    for ip in cachedIPs {
+                    for ip in cachedIPs where !seenDestinations.contains(ip) {
+                        seenDestinations.insert(ip)
                         routesToAdd.append((destination: ip, gateway: gateway, isNetwork: false, source: result.source))
                     }
                 } else {
@@ -991,7 +994,8 @@ final class RouteManager: ObservableObject {
 
         // Collect IP ranges (these don't need DNS resolution)
         for service in config.services where service.enabled {
-            for range in service.ipRanges {
+            for range in service.ipRanges where !seenDestinations.contains(range) {
+                seenDestinations.insert(range)
                 routesToAdd.append((destination: range, gateway: gateway, isNetwork: true, source: service.name))
             }
         }
@@ -1094,12 +1098,14 @@ final class RouteManager: ObservableObject {
 
         var newRoutes: [ActiveRoute] = []
         var routesToAdd: [(destination: String, gateway: String, isNetwork: Bool, source: String)] = []
+        var seenDestinations: Set<String> = []  // Deduplicate by destination IP
 
         // Build routes from DNS cache
         for service in config.services where service.enabled {
             for domain in service.domains {
                 if let cachedIPs = dnsDiskCache[domain] {
-                    for ip in cachedIPs {
+                    for ip in cachedIPs where !seenDestinations.contains(ip) {
+                        seenDestinations.insert(ip)
                         routesToAdd.append((destination: ip, gateway: gateway, isNetwork: false, source: service.name))
                     }
                     if let firstIP = cachedIPs.first {
@@ -1108,15 +1114,17 @@ final class RouteManager: ObservableObject {
                 }
             }
             // IP ranges don't need DNS
-            for range in service.ipRanges {
+            for range in service.ipRanges where !seenDestinations.contains(range) {
+                seenDestinations.insert(range)
                 routesToAdd.append((destination: range, gateway: gateway, isNetwork: true, source: service.name))
             }
         }
 
         // Custom domains
         for domain in config.domains where domain.enabled {
             if let cachedIPs = dnsDiskCache[domain.domain] {
-                for ip in cachedIPs {
+                for ip in cachedIPs where !seenDestinations.contains(ip) {
+                    seenDestinations.insert(ip)
                     routesToAdd.append((destination: ip, gateway: gateway, isNetwork: false, source: domain.domain))
                 }
                 if let firstIP = cachedIPs.first {
@@ -1579,6 +1587,10 @@ final class RouteManager: ObservableObject {
         var newRoutes: [ActiveRoute] = []
         var routesToAdd: [(destination: String, gateway: String, isNetwork: Bool)] = []
 
+        // Track existing routes to avoid duplicates
+        let existingDestinations = Set(activeRoutes.map { $0.destination })
+        var seenDestinations: Set<String> = existingDestinations
+        
         // Capture DNS settings for parallel resolution
         let userDNS = detectedDNSServer
         let fallbackDNS = config.fallbackDNS
@@ -1608,14 +1620,16 @@ final class RouteManager: ObservableObject {
                 dnsCache[domain] = firstIP
             }
 
-            for ip in ips {
+            for ip in ips where !seenDestinations.contains(ip) {
+                seenDestinations.insert(ip)
                 routesToAdd.append((destination: ip, gateway: gateway, isNetwork: false))
                 newRoutes.append(ActiveRoute(destination: ip, gateway: gateway, source: service.name, timestamp: Date()))
             }
         }
 
         // Add IP ranges (no DNS needed)
-        for range in service.ipRanges {
+        for range in service.ipRanges where !seenDestinations.contains(range) {
+            seenDestinations.insert(range)
             routesToAdd.append((destination: range, gateway: gateway, isNetwork: true))
             newRoutes.append(ActiveRoute(destination: range, gateway: gateway, source: service.name, timestamp: Date()))
         }

Sources/SettingsView.swift

@@ -1263,7 +1263,7 @@ struct GeneralTab: View {
             HStack {
                 VStack(alignment: .leading, spacing: 2) {
                     BrandedAppName(fontSize: 13)
-                    Text("Version 1.6.0")
+                    Text("Version 1.6.1")
                         .font(.system(size: 11))
                         .foregroundColor(Color(hex: "6B7280"))
                 }
@@ -1721,7 +1721,7 @@ struct InfoTab: View {
             // App name with branded colors
             BrandedAppName(fontSize: 24)
 
-            Text("v1.6.0")
+            Text("v1.6.1")
                 .font(.system(size: 12, design: .monospaced))
                 .foregroundColor(Color(hex: "6B7280"))
 

docs/CHANGELOG.md

@@ -5,6 +5,11 @@ All notable changes to VPN Bypass will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.6.1] - 2026-01-21
+
+### Fixed
+- **Deduplicate Routes** - Multiple domains resolving to the same IP no longer create duplicate routes
+
 ## [1.6.0] - 2026-01-21
 
 ### Added