Tunnel edit: While preparing for reuse, should make onValueBeingEdited nil as well

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

Version bump

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

iPad: Ensure we set sourceRect for all cases where we use sourceView

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

Info.plist: Register for handling public.text files for Open-in

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

Importing: Also support importing public.text files in the file picker

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

iPad: Set correct sourceRect for the popover anchored on the central 'Add' button

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

TunnelsManager: Return a manager with no tunnels in the simulator

To be able to run at least parts of the app in the simulator.

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

iPad: Configuring the split-view controller should happen in init(), not loadView()

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

Settings: show build id

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

Make license consistent

We changed all the files and the README to MIT a long time ago but
forgot to update COPYING.

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

Version bump

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

Importing: simplify

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

Importing: Assume imported files without .conf or .zip extensions to be a config file

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

DNSResolver: No need to resolve if the endpoint is already an IP address

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

Model: Endpoint.hasHostAsIPAddress()

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

PacketTunnelProvider: modernize header

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

Not horribly broken

Instead it's just mostly broken. Maybe someday it will only be partially
broken. Then a bit broken. And then maybe not broken at all? Before, of
course, it's broken again.

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

Global: swiftlint autocorrect --format

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

Global: swiftlint autocorrect

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

Importing: Error out on file with unsupported file extension

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

wireguard-go-bridge: take fd instead of fnptr

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

Xcode: enable more warnings

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

Tunnel edit: TunnelEditTableViewKeyValueCell need not support a read-only mode now

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

Tunnel edit: A new cell class for the public key field, to make the value scrollable

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

Tunnel detail: Refactor out the label scrolling into a separate UI class

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

Tunnel view model: Peers in a configuation may not share the same public key

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

Parser: Peers in a configuation may not share the same public key

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

Version bump

We update the build number, not the version number, so that test flight
doesn't need a new review.

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

Parser: Address, DNS and AllowedIPs can have mutliple entries

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

QR Code: Dismiss the QR code screen when the name prompt is cancelled

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

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>