control/controlclient: use API to get Windows version number

Fixes #1478

Signed-off-by: Aleksandar Pesic <peske.nis@gmail.com>
This commit is contained in:
Aleksandar Pesic
2021-03-13 01:30:26 +01:00
committed by Brad Fitzpatrick
parent 2bc518dcb2
commit 4224b3f731
2 changed files with 33 additions and 15 deletions
@@ -0,0 +1,28 @@
// Copyright (c) 2021 Tailscale Inc & AUTHORS All rights reserved.
// Use of this source code is governed by a BSD-style
// license that can be found in the LICENSE file.
package controlclient
import (
"os/exec"
"regexp"
"testing"
)
func TestOsVersionWindows(t *testing.T) {
out, err := exec.Command("cmd", "/c", "ver").Output()
if err != nil {
t.Fatalf("`ver` error: %v", err)
}
// Extract the version number from the output, and isolate the first three parts (major.minor.build)
rx := regexp.MustCompile(`(\d+\.\d+\.\d+)(\.\d+)?`)
m := rx.FindStringSubmatch(string(out))
if m == nil {
t.Fatalf("no version number in `ver` output: %q", out)
}
got := osVersionWindows()
if m[1] != got {
t.Errorf("osVersionWindows got %q want %q", got, m[1])
}
}