From: Roopesh Chander <roop@roopc.net>
Date: Sat, 22 Dec 2018 10:43:31 +0000 (+0530)
Subject: Don't set username on NETunnelProviderProtocol
X-Git-Tag: 0.0.20181225-1~10
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=d29f47fc9b01f71fcf6eaf021ee5a4c2a9b9a939;p=thirdparty%2Fwireguard-apple.git

Don't set username on NETunnelProviderProtocol

The username corresponds to the Account field in iOS system VPN UI,
but if we don't set it, the field is not shown, so setting it isn't
really required.

Signed-off-by: Roopesh Chander <roop@roopc.net>
---

diff --git a/WireGuard/Shared/Model/NETunnelProviderProtocol+Extension.swift b/WireGuard/Shared/Model/NETunnelProviderProtocol+Extension.swift
index d2179e0..df3fd04 100644
--- a/WireGuard/Shared/Model/NETunnelProviderProtocol+Extension.swift
+++ b/WireGuard/Shared/Model/NETunnelProviderProtocol+Extension.swift
@@ -32,9 +32,6 @@ extension NETunnelProviderProtocol {
         } else {
             serverAddress = "Multiple endpoints"
         }
-
-        //TODO(roopc): Why are we doing this? Just for kicks? Is it useful? Seems needless.
-        username = tunnelConfiguration.name
     }
 
     func asTunnelConfiguration(called name: String? = nil) -> TunnelConfiguration? {