aboutsummaryrefslogtreecommitdiffstats
path: root/WireGuard (follow)
Commit message (Collapse)AuthorAgeFilesLines
* Most similar views now shared between ViewControllersEric Kuck2018-12-1428-506/+278
| | | | Signed-off-by: Eric Kuck <eric@bluelinelabs.com>
* Prettier log time formatJason A. Donenfeld2018-12-151-9/+19
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* Simplify logging tagsJason A. Donenfeld2018-12-146-22/+31
| | | | | | | This was roop's initial idea, and it turns out to be the better one, now that we can pass cstrings more easily. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* Fix tunnel remaining in 'Activating' stateRoopesh Chander2018-12-141-1/+19
| | | | | | | | It uses to remain in 'Activating' state when we don't get a status update notification, for example, when turning on the tunnel repeatedly without Internet connectivity. Signed-off-by: Roopesh Chander <roop@roopc.net>
* Fix status switch weird state after an error occursRoopesh Chander2018-12-142-2/+2
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* Deduplicate functionsJason A. Donenfeld2018-12-142-10/+2
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* Simplify filemanager extensionJason A. Donenfeld2018-12-131-18/+13
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* Can't -> cannotJason A. Donenfeld2018-12-132-6/+6
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* NE: Communicate last error to app through a shared fileRoopesh Chander2018-12-144-16/+84
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* TunnelsManager: startActivation() need not take a tunnelConfigurationRoopesh Chander2018-12-141-11/+3
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* Localize swiftlintJason A. Donenfeld2018-12-131-1/+2
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* Reorganized ViewControllers (split out UIViews and UITableViewCells into their own classes)Eric Kuck2018-12-1331-719/+1360
| | | | | | All swiftlint warnings except one fixed up Signed-off-by: Eric Kuck <eric@bluelinelabs.com>
* Avoid escaping heap allocationJason A. Donenfeld2018-12-131-3/+3
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* Tidy up str to gostr conversionJason A. Donenfeld2018-12-131-16/+8
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* TunnelsManager: Remove mentions of 'internal error'Roopesh Chander2018-12-131-5/+5
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* TunnelsManager: Ask to check Internet connectivity in error alertRoopesh Chander2018-12-131-1/+1
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* Remove unused code: InternetReachabilityRoopesh Chander2018-12-132-55/+0
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* TunnelsManager: Remove unused variableRoopesh Chander2018-12-131-4/+0
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* TunnelsManager: Remove assertRoopesh Chander2018-12-131-1/+0
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* Simplify C stringsJason A. Donenfeld2018-12-131-13/+6
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* TunnelsManager: Fix race between multiple startActivation() callsRoopesh Chander2018-12-131-0/+2
| | | | | | | | After startActivate() is called on a waiting tunnel, user might turn on a different tunnel before the waiting tunnel's status gets updated. This fix prevents that from happening. Signed-off-by: Roopesh Chander <roop@roopc.net>
* TunnelsManager: Reintroduce waiting for another tunnel to deactivateRoopesh Chander2018-12-132-3/+23
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* Rewrite LoggerJason A. Donenfeld2018-12-136-114/+86
| | | | | | | This reverts all of Roop's changes to the C code, and then rewrites the logger logic to be cleaner. Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* More reliable logo sizingEric Kuck2018-12-131-33/+34
| | | | Signed-off-by: Eric Kuck <eric@bluelinelabs.com>
* TunnelsManager: Report activation errors through the activationDelegateRoopesh Chander2018-12-134-68/+77
| | | | | | Don't report activation errors through completion handlers Signed-off-by: Roopesh Chander <roop@roopc.net>
* Logging: Write versions from both app and extensionRoopesh Chander2018-12-133-17/+13
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* Logging: file_log doesn't need the message typeRoopesh Chander2018-12-131-3/+3
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* Logging: Tag the entries in the merged logRoopesh Chander2018-12-134-7/+14
| | | | | | | So we know which entry is from the app and which is from the network extension. Signed-off-by: Roopesh Chander <roop@roopc.net>
* Logging: Use ringlogger for logging from the appRoopesh Chander2018-12-133-22/+23
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* Logging: ringlogger.c: Trim trailing newlinesRoopesh Chander2018-12-131-0/+7
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* Logging: Use ringlogger for logging from the extensionRoopesh Chander2018-12-1310-69/+121
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* Avoid using 'VPN' in code where possibleRoopesh Chander2018-12-132-19/+19
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* mv WireGuard/WireGuard/VPN/ WireGuard/WireGuard/Tunnel/Roopesh Chander2018-12-134-3/+3
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* Commit untested ringlogger codeJason A. Donenfeld2018-12-132-0/+139
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* Remove useless whitespaceJason A. Donenfeld2018-12-136-34/+34
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* Settings: better padding calculationJason A. Donenfeld2018-12-131-3/+3
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* Fixed hacky logo display for settingsEric Kuck2018-12-121-3/+9
| | | | Signed-off-by: Eric Kuck <eric@bluelinelabs.com>
* More formatting nits and cyclomatic complexity fixesEric Kuck2018-12-1217-217/+166
| | | | Signed-off-by: Eric Kuck <eric@bluelinelabs.com>
* More linter warnings fixed, enabled more swiftlint rules, project cleanupEric Kuck2018-12-1221-521/+572
| | | | Signed-off-by: Eric Kuck <eric@bluelinelabs.com>
* Tons more swiftlint warnings fixed. Still a few remaining.Eric Kuck2018-12-1221-237/+222
| | | | Signed-off-by: Eric Kuck <eric@bluelinelabs.com>
* Added swiftlint and fixed all errors (and a bunch, but not all, warnings)Eric Kuck2018-12-1223-394/+431
| | | | Signed-off-by: Eric Kuck <eric@bluelinelabs.com>
* Zip importing: importFromFile should take a completionHandlerRoopesh Chander2018-12-122-7/+13
| | | | | | | Deletion of the being-imported file should be done in the completionHandler. Signed-off-by: Roopesh Chander <roop@roopc.net>
* ErrorPresenter: Support onPresented for showErrorAlert(title:,message:)Roopesh Chander2018-12-121-3/+4
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>
* Zip importing: Handle spaces in filenames correctlyRoopesh Chander2018-12-122-13/+14
| | | | | | | Previously, if a filename of a .conf file inside the zip file contained spaces, it was not imported. Signed-off-by: Roopesh Chander <roop@roopc.net>
* Supply missing pieces of path changeJason A. Donenfeld2018-12-122-10/+12
| | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>
* Fixed a potential race condition, better naming on PacketTunnelSettingsGenerator methodsEric Kuck2018-12-112-18/+24
| | | | Signed-off-by: Eric Kuck <eric@bluelinelabs.com>
* Added an (unfinished) NWPathMonitor implementation for reconnecting on network changesEric Kuck2018-12-112-1/+39
| | | | Signed-off-by: Eric Kuck <eric@bluelinelabs.com>
* State restoration: Don't create duplicate mainVC and tunnelsListVCRoopesh Chander2018-12-111-6/+1
| | | | | | | | | | | | | | | | | | | This creates a duplicate tunnels manager, leading to problems tracking tunnel statuses. To reproduce the bug that this commit fixes, you can do the following: 1. Remove all tunnels 2. Run in Xcode 3. Import zip with ~10 tunnels 4. Stop app in Xcode 5. Run in Xcode 6. Turn on 1st tunnel, after it's on, turn off Turn on 2nd tunnel, after it's on, turn off ... After 6-8 tunnels, the spinner doesn't show up, indicating that the status is not being tracked. Signed-off-by: Roopesh Chander <roop@roopc.net>
* TunnelsManager: Error out only on no-internet scenarioRoopesh Chander2018-12-111-4/+4
| | | | | | The other scenario happens even during reloading of a tunnel for activation. Signed-off-by: Roopesh Chander <roop@roopc.net>
* Remove the feature of waiting for another tunnel to deactivateRoopesh Chander2018-12-113-59/+11
| | | | Signed-off-by: Roopesh Chander <roop@roopc.net>