Tunnel view model: No need to save if we already have a validated configuration object

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

global: convert to Swift 4.2

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

TunnelsManager: account for no or many endpoints

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

wireguard-go-bridge: work around DWARF generation issue

Golang bug 25148 upstream.

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

PacketTunnelProvider: clamp v6 addresses to /120

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Settings: account for bottom padding on iPhone X

Reported-by: Jeffrey A. Donenfeld <jad@runblue.com>
Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

XCode: use old network extension ID

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

wireguard-go-bridge: xcode expects install target

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

XCode: unify version strings

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Settings: anchor logo to bottom

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Add a correct launch-screen storyboard

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

Revert "Settings: Swap table view sections"

This reverts commit 1108f8248946f80d29012d8d43658db8461c0fed.

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Settings: simplify

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Global: run through swiftlint

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Settings: limit size of logo

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Settings: use updated wireguard banner logo

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

TunnelsManager: restore sanity

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Change version with 0.0.yyyymmdd versioning scheme

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

VPN: Make sure actions inside tunnelProvider completion handlers are done in the main queue

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

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

But we do, which we'll fix subsequently

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

Revert "TunnelsManager: initialize observer on correct thread"

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>

Settings: Show the captioned WireGuard logo at the end

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

Settings: Swap table view sections

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

Remove empty image

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

Add Wireguard logo with caption to project

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Tunnels list: Fix status switch look after flipping it off after an error

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

ErrorPresenter: showErrorAlert takes an onPresentated handler

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

Tunnels list: Should deselect the row when handling the selection

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

Tunnel list: Don't show the detail view after creating a config from scratch

So it's consistent with the other two "Add a configuration" menu items

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

Ensure that all tunnel names are trimmed of whitespaces

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

Zip archive: Handle files within folders inside the zip

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

Update readme

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

TunnelsManager: initialize observer on correct thread

Because of the callback recursion, this function needs to always be called from the right thread

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

TunnelsManager: do not track currentTunnel

It's error prone and does not give us anything.

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

TunnelsManager: do not change back to inactive if we are resolving the endpoint

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Tunnels list: Remove separator lines in the table view

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

Tunnels list: Show an 'Add' button when there are no tunnels

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

Tunnels list: Show a busy indicator before the tunnels are loaded

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

Tunnels list: The Add and Settings buttons should be no-ops until the tunnels are loaded

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

Tunnel edit: Use layoutMarginsGuide instead of constants

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

TunnelsManager: fix use of arrayslice in addMultiple

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

TunnelsManager: get rid of name set

Instead, tunnels itself should be a sorted-set. For now it's a list,
which is probably fine though.

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

TunnelsManager: get rid of index management

No need for premature optimization. There aren't that many tunnels most
of the time, and calling sort on a partially sorted array is fast.

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Global: no periods at the end of error messages

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Import: rework addMultiple logic

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Tunnel detail: In the key-value cell, make the value label scrollable horizontally

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

Tunnel edit: Ensure all key-value fields have a placeholder text

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

DNSResolver: Bugfix: Always call completion handler

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

Info.plist: Update document icons and description

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

Info.plist: Document type UTI should be consistent with exported UTI

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

Info.plist: Register zip file for open-in

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

Info.plist: Add .conf extension to document type

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

Add document icon

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

Add app icons

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

Remove files we don't need

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

Handle cell reuse in showing delete buttons in red

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

Table edit: Show/hide 'Exclude Private IPs' instead of enable/disable

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

Tunnel view model: If there's just one peer, populate the scratchpad on init

So that exclude private IPs can be computed correctly.

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

Tunnel view model: Initialize peerData.numberOfPeers

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

DNSResolver: getaddrinfo returns a sockaddr and inet_ntop takes strlen

The way this was written before was totally wrong.

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

wireguard-go-bridge: use boottime instead of monotonic

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

NetworkExtension: check isTunnelClosed before packet existence

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Global: OK over Ok

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Deletion buttons are shown in red

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

Tunnel edit: Highlight erroring field keys in red

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

Tunnels list: Swipe left to delete

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

Global: fix up strings

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

QR code: Fix what happens when an error is shown

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

NE: Show an alert when someone tries to start the VPN from iOS Settings

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

Export: fix up export format

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

Tunnel errors: Consolidate tunnel activation errors into ErrorPresenter

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

TunnelsManager: Better errors

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

ConfigFile: WgQuickConfigFileWriter: Fix '[Peer]' section header

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

Importing: Account for errors in showing the 'No configurations found' error

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

Tunnel detail: Fix deletion of tunnel

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

VPN: TunnelsManager: Fix deletion code

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

VPN: Restart tunnel after modifying a currently active configuration

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

QR code: Degrade gracefully when running on the simulator or on a device without a camera

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

Revert "QR code: QR code scanning is not available in the simulator"

This reverts commit 9729386406cd9543303a68e51d57b286880bb8cd.

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

Importing: Show error when the zip file contains no .conf files

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

VPN: We have to start observing the tunnelContainer before we attempt to activate it

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

Importing: Error out on duplicate name

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

QR Code: Error out on duplicate name

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

Tunnel edit: Error out on duplicate name when creating / modifying a tunnel

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

VPN: An ErrorPresenter class to consolidate showing of error alerts

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

VPN: Introduce an error enum for adding/modifying/removing tunnels

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

VPN: TunnelsManager: s/TunnelsManagerError/TunnelActivationError/g;

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

VPN: TunnelsManager should keep track of tunnel names to prevent duplicate names

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

UI: not all subclasses of CopyableLabelTableViewCell have a detailTextLabel, so generalize

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

TunnelManager: bound recursion in startActivation

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

VPN: Retry startTunnel based on error conditions

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

Tunnel list: Fix importing zip with multiple configurations

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

VPN: Support adding multiple configurations in one shot

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

wireguard-go-bridge: extract version for consumption by network extension

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

wireguard-go-bridge: simplify logging and remove tags

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

NetworkExtension: close context before turning off go

The go turnOff code waits for all readers to have completed. In order to
make readers complete, we must send the signal to unblock the request.
Thus, we must reorder these.

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

wireguard-go-bridge: handle errors in makefile

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

UI: More elegant copy-to-clipboard behavior

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>

wireguard-go: pull in recent change

Signed-off-by: Jason A. Donenfeld <Jason@zx2c4.com>