Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | mv WireGuard/WireGuard/VPN/ WireGuard/WireGuard/Tunnel/ | Roopesh Chander | 2018-12-13 | 3 | -606/+0 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Remove useless whitespace | Jason A. Donenfeld | 2018-12-13 | 1 | -5/+5 |
| | | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> | ||||
* | More formatting nits and cyclomatic complexity fixes | Eric Kuck | 2018-12-12 | 1 | -37/+34 |
| | | | | Signed-off-by: Eric Kuck <eric@bluelinelabs.com> | ||||
* | More linter warnings fixed, enabled more swiftlint rules, project cleanup | Eric Kuck | 2018-12-12 | 2 | -20/+17 |
| | | | | Signed-off-by: Eric Kuck <eric@bluelinelabs.com> | ||||
* | Tons more swiftlint warnings fixed. Still a few remaining. | Eric Kuck | 2018-12-12 | 3 | -40/+39 |
| | | | | Signed-off-by: Eric Kuck <eric@bluelinelabs.com> | ||||
* | Added swiftlint and fixed all errors (and a bunch, but not all, warnings) | Eric Kuck | 2018-12-12 | 1 | -25/+24 |
| | | | | Signed-off-by: Eric Kuck <eric@bluelinelabs.com> | ||||
* | TunnelsManager: Error out only on no-internet scenario | Roopesh Chander | 2018-12-11 | 1 | -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 deactivate | Roopesh Chander | 2018-12-11 | 1 | -55/+9 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | TunnelsManager: Deactivate only when the status becomes 'connected' | Roopesh Chander | 2018-12-11 | 1 | -3/+11 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | TunnelsManager: Handle status change in TunnelsManager | Roopesh Chander | 2018-12-11 | 1 | -66/+78 |
| | | | | | | Rather than in TunnelContainer. Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Error handling: alertText() can be nil | Roopesh Chander | 2018-12-11 | 1 | -1/+1 |
| | | | | | | Indicating that no alert is to be shown for that error. Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | TunnelsManager: Don't act on status change on tunnelProviders we don't have | Roopesh Chander | 2018-12-10 | 1 | -2/+0 |
| | | | | | | That causes errors we don't want, and duplicate notifications. Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | TunnelsManager: Always call the completion handler before returning | Roopesh Chander | 2018-12-10 | 1 | -0/+4 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | TunnelsManager: Debugging helpers for tunnel status | Roopesh Chander | 2018-12-10 | 1 | -0/+27 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | TunnelsManager: Handle deactivation of a waiting tunnel | Roopesh Chander | 2018-12-10 | 1 | -1/+6 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | TunnelsManager: Observe status for all tunnels in one block | Roopesh Chander | 2018-12-08 | 1 | -52/+54 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | TunnelsManager: Remove unused variables | Roopesh Chander | 2018-12-08 | 1 | -14/+0 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | State restoration: Restore tunnel detail view | Roopesh Chander | 2018-12-07 | 1 | -0/+4 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Error handling: Introduce a WireGuardResult type to handle errors in callbacks across the app | Roopesh Chander | 2018-12-07 | 1 | -28/+2 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Error handling: Introduce a WireGuardAppError protocol to manage errors | Roopesh Chander | 2018-12-07 | 1 | -1/+27 |
| | | | | | | The alert strings shall be located next to where the errors are declared. Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Error handling: Cleanup Tunnels Manager errors | Roopesh Chander | 2018-12-07 | 1 | -45/+65 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Refactor out VPN-handling stuff from tunnels list VC to the main VC | Roopesh Chander | 2018-12-03 | 1 | -8/+8 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | VPN: Error out when tunnel activation fails because there's no internet | Roopesh Chander | 2018-12-03 | 2 | -6/+91 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | It's 'WiFi', not 'Wifi' | Roopesh Chander | 2018-11-28 | 1 | -6/+6 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Tunnels manager: After saving after activating on-demand, reload tunnel | Roopesh Chander | 2018-11-14 | 1 | -3/+16 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Tunnels manager: Need to keep VPN-on-demand tunnels's status under observation | Roopesh Chander | 2018-11-12 | 1 | -4/+13 |
| | | | | | | Because they can turn on automatically, even while the app is in the foreground. Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Tunnels manager: Keep track of NETunnelProviderManager's isOnDemandEnabled property | Roopesh Chander | 2018-11-12 | 1 | -3/+8 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Model, Tunnels manager: Rewrite the model for VPN-on-demand | Roopesh Chander | 2018-11-12 | 2 | -36/+86 |
| | | | | | | | | | The VPN-on-demand settings should not be part of the tunnel configuration. Rather, the onDemandRules stored in the tunnel provider configuration serve as the one place where the VPN-on-demand settings are stored. Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | TunnelsManager: Support for on-demand rules | Roopesh Chander | 2018-11-11 | 1 | -0/+36 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | VPN: When activating while another tunnel is active, deactivate the other tunnel | Roopesh Chander | 2018-11-10 | 1 | -7/+14 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | TunnelsManager: startDeactivation() need not take a completion handler | Roopesh Chander | 2018-11-10 | 1 | -2/+1 |
| | | | | | | Because the completion handler pattern doesn't fit in this case. Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | VPN: There are no DNS errors to handle in the app now | Roopesh Chander | 2018-11-10 | 1 | -1/+0 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | TunnelsManager: Ability to refresh connection statuses | Roopesh Chander | 2018-11-09 | 1 | -0/+14 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Move logic to extension: NETunnelProviderProtocol extension code should be shared | Roopesh Chander | 2018-11-08 | 1 | -31/+0 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Move logic to extension: Move DNSResolver to extension | Roopesh Chander | 2018-11-08 | 1 | -158/+0 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Move logic to extension: .resolvingEndpointDomains is not longer a valid status | Roopesh Chander | 2018-11-08 | 1 | -5/+0 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Move logic to extension: DNS resolution no longer happens in the app | Roopesh Chander | 2018-11-08 | 1 | -44/+5 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Move logic to extension: Move PacketTunnelOptionsGenerator to the extension | Roopesh Chander | 2018-11-08 | 1 | -178/+0 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Move logic to extension: Invoke startTunnel() without any options | Roopesh Chander | 2018-11-08 | 1 | -4/+1 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | TunnelsManager: Return a manager with no tunnels in the simulator | Roopesh Chander | 2018-11-07 | 1 | -0/+5 |
| | | | | | | To be able to run at least parts of the app in the simulator. Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | DNSResolver: No need to resolve if the endpoint is already an IP address | Roopesh Chander | 2018-11-06 | 1 | -2/+6 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Global: swiftlint autocorrect --format | Roopesh Chander | 2018-11-06 | 1 | -6/+6 |
| | | | | Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | TunnelsManager: account for no or many endpoints | Jason A. Donenfeld | 2018-11-05 | 2 | -20/+20 |
| | | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> | ||||
* | XCode: use old network extension ID0.0.20181104-1 | Jason A. Donenfeld | 2018-11-04 | 1 | -1/+1 |
| | | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> | ||||
* | Global: run through swiftlint | Jason A. Donenfeld | 2018-11-03 | 3 | -12/+12 |
| | | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> | ||||
* | TunnelsManager: restore sanity | Jason A. Donenfeld | 2018-11-03 | 1 | -45/+20 |
| | | | | Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com> | ||||
* | VPN: Make sure actions inside tunnelProvider completion handlers are done in the main queue | Roopesh Chander | 2018-11-03 | 1 | -17/+35 |
| | | | | | | They should really already be in the main queue, but we do this just in case. Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | VPN: Start observing the status only if startTunnel() succeeds | Roopesh Chander | 2018-11-03 | 1 | -1/+1 |
| | | | | | | This should fix the status change to .disconnected when we're resolving endpoints Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | VPN: We should never get a status change to .disconnected if we're resolving endpoints | Roopesh Chander | 2018-11-03 | 1 | -0/+1 |
| | | | | | | But we do, which we'll fix subsequently Signed-off-by: Roopesh Chander <roop@roopc.net> | ||||
* | Revert "TunnelsManager: initialize observer on correct thread" | Roopesh Chander | 2018-11-03 | 1 | -7/+5 |
| | | | | | | | | | Rather, we'll make sure we always call startObservingTunnelStatus from the main thread. This reverts commit d16a6a167c8217e81dd9111d1ad9e4ec6429ef3b. Signed-off-by: Roopesh Chander <roop@roopc.net> |