Development discussion of WireGuard
 help / color / mirror / Atom feed
From: "Jason A. Donenfeld" <Jason@zx2c4.com>
To: Joshua Sjoding <joshua.sjoding@scjalliance.com>
Cc: WireGuard mailing list <wireguard@lists.zx2c4.com>
Subject: Re: WireGuard for Windows failed to start after update to v0.2.1
Date: Tue, 17 Nov 2020 22:56:29 +0100	[thread overview]
Message-ID: <CAHmME9oZvNQjCoShdMTQ2n0soTA0wNVVm9rPFcma0fP-6_9tsw@mail.gmail.com> (raw)
In-Reply-To: <CALaUSuuzQhTcDE_b6PPf=fXuOjbmVf7Tgzdz9DbWMxCJ8-2WUQ@mail.gmail.com>

Hi Joshua,

On Tue, Nov 17, 2020 at 10:20 PM Joshua Sjoding
<joshua.sjoding@scjalliance.com> wrote:
>
> I think they're separate issues both related to the new version, which
> is why I opted to send in separate emails. The other email is about an
> issue first encountered on two other computers within the company.
>
> In my case I was able to get WireGuard working again. Here's what happened:
>
> 1. I manually started the WireGuard desktop app. It immediately
> prompted for elevated privileges, which is atypical. I provided it
> with administrative credentials.
> 2. The WireGuard desktop app launched, but lacked any of my configured
> tunnels and still was prompting me to upgrade. I see that there's a
> config data migration recorded in the log, so my guess is that
> WireGuard was still running the old executable and didn't know to look
> for the configuration data in the new location yet.
> 3. I told WireGuard to run the updater again and this time it went through fine.
> 4. When WireGuard came back up it had the tunnel configuration again.
> The tunnel was left in a shut off state, even though it was on prior
> to the first upgrade attempt. I switched the tunnel back on and it
> seems to be working fine now.

It's unclear to me where the timeline begins in this one. Here's my
present understanding:

1. It's a normal Tuesday at SCJ, the coffee is roasting, the pens are
clicking, when WireGuard innocently informs you that there's an update
available.

2. You click update, it downloads the installer, and runs it. After
hanging for a bit, the installer seems to run in reverse, and you're
kicked back to the WireGuard app, not-updated, except all your config
files are gone, which is weird.

3. You click update again. It succeeds, and the new WireGuard version
comes up, and your configs are all there.

4. The tunnels aren't started, so you press start, and everything
works, but there are these messages in your log, which you sent me in
the other thread.

Is that an accurate story?

If so, my current best guess is that:
- MSI stops manager, stops tunnel, installs new version, starts new
manager (thereby initiating the config migration), starts new tunnel.
- New tunnel fails to start [a]. We flipped on the MSI rollback switch
in v0.2, so that tunnel bug now causes the installer to move in
reverse [b].
- New manager stopped, old version installed, old manager started, old
tunnel started, but config has been migrated already [c], so the
tunnel doesn't come up and you can't see it.
- You press update again, stops manager, new version installed, starts manager.
- You start the tunnel service manually.

Current status based on the above is:
- [a] and [c] were fixed by
https://git.zx2c4.com/wireguard-windows/commit/?id=2edb9007c820656514a0e9b07c9f83ad86593866
and https://go-review.googlesource.com/c/sys/+/270897 .
- [c] is fixed by
https://git.zx2c4.com/wireguard-windows/commit/?id=2edb9007c820656514a0e9b07c9f83ad86593866
.
- [b] we can either fix by getting rid of rollback again, or keeping
it, and arguing that [c] is the more proper fix.

Can you confirm this corresponds?

Jason

  reply	other threads:[~2020-11-17 21:57 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-11-17 17:46 Joshua Sjoding
2020-11-17 20:59 ` Jason A. Donenfeld
2020-11-17 21:19   ` Joshua Sjoding
2020-11-17 21:56     ` Jason A. Donenfeld [this message]
2020-11-17 22:03       ` Jason A. Donenfeld
2020-11-17 22:05         ` Jason A. Donenfeld
2020-11-17 22:46       ` Joshua Sjoding
2020-11-18  0:05         ` Jason A. Donenfeld
2020-11-18  0:42           ` Jason A. Donenfeld
2020-11-18 13:41             ` Jason A. Donenfeld
2020-11-19 19:39               ` Joshua Sjoding
2020-11-19 20:00                 ` Jason A. Donenfeld

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAHmME9oZvNQjCoShdMTQ2n0soTA0wNVVm9rPFcma0fP-6_9tsw@mail.gmail.com \
    --to=jason@zx2c4.com \
    --cc=joshua.sjoding@scjalliance.com \
    --cc=wireguard@lists.zx2c4.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).