From 8f8abcdade7cf92e7ae706ef1ea207d033e91fb4 Mon Sep 17 00:00:00 2001
From: Davide De Rosa <keeshux@gmail.com>
Date: Sat, 27 Jan 2024 10:27:46 +0100
Subject: [PATCH] Opt out of platform info in peer info (#409)

---
 CHANGELOG.md                                  |  6 ++++
 .../CoreConfiguration+OpenVPN.swift           | 28 +++++++++++--------
 2 files changed, 23 insertions(+), 11 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 55c8a5b..f6fbf8f 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## Unreleased
+
+### Changed
+
+- OpenVPN: Opt out of platform info in peer info. [#409](https://github.com/passepartoutvpn/tunnelkit/pull/409)
+
 ## 6.3.2 (2024-01-05)
 
 ### Fixed
diff --git a/Sources/TunnelKitOpenVPNProtocol/CoreConfiguration+OpenVPN.swift b/Sources/TunnelKitOpenVPNProtocol/CoreConfiguration+OpenVPN.swift
index 804d369..9a8f5c2 100644
--- a/Sources/TunnelKitOpenVPNProtocol/CoreConfiguration+OpenVPN.swift
+++ b/Sources/TunnelKitOpenVPNProtocol/CoreConfiguration+OpenVPN.swift
@@ -64,18 +64,10 @@ extension CoreConfiguration {
 
         // MARK: Authentication
 
-        static func peerInfo(extra: [String: String]? = nil) -> String {
-            let platform: String
-            let platformVersion = ProcessInfo.processInfo.operatingSystemVersion
-            #if os(iOS)
-            platform = "ios"
-            #else
-            platform = "mac"
-            #endif
+        static func peerInfo(withPlatform: Bool = true, extra: [String: String]? = nil) -> String {
             let uiVersion = versionIdentifier ?? "\(identifier) \(version)"
             var info = [
                 "IV_VER=2.4",
-                "IV_PLAT=\(platform)",
                 "IV_UI_VER=\(uiVersion)",
                 "IV_PROTO=2",
                 "IV_NCP=2",
@@ -89,10 +81,24 @@ extension CoreConfiguration {
 //            if pushPeerInfo {
             if true {
                 info.append("IV_SSL=\(CryptoBox.version())")
+            }
+            if withPlatform {
+                let platform: String
+                let platformVersion = ProcessInfo.processInfo.operatingSystemVersion
+#if os(iOS)
+                platform = "ios"
+#elseif os(tvOS)
+                platform = "tvos"
+#else
+                platform = "mac"
+#endif
+                info.append("IV_PLAT=\(platform)")
                 info.append("IV_PLAT_VER=\(platformVersion.majorVersion).\(platformVersion.minorVersion)")
             }
-            if let extra = extra {
-                info.append(contentsOf: extra.map { "\($0)=\($1)" })
+            if let extra {
+                info.append(contentsOf: extra.map {
+                    "\($0)=\($1)"
+                })
             }
             info.append("")
             return info.joined(separator: "\n")