Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

cmd/tailscale/cli: make 'tailscale update' support Debian/Ubuntu apt #7028

Merged
merged 1 commit into from
Jan 20, 2023

Conversation

bradfitz
Copy link
Member

Updates #6995

cmd/tailscale/cli/update.go Outdated Show resolved Hide resolved
@bradfitz bradfitz force-pushed the bradfitz/update_deb branch 2 times, most recently from 63669e5 to 34be0f6 Compare January 20, 2023 19:53
cmd/tailscale/cli/update.go Fixed Show fixed Hide fixed
cmd/tailscale/cli/update.go Fixed Show fixed Hide fixed
bs := bufio.NewScanner(bytes.NewReader(was))
hadCorrect := false
commentLine := regexp.MustCompile(`^\s*\#`)
pkgsURL := regexp.MustCompile(`\bhttps://pkgs\.tailscale\.com/((un)?stable)/`)

Check failure

Code scanning / CodeQL

Missing regular expression anchor

When this is used as a regular expression on a URL, it may match anywhere, and arbitrary hosts may come before or after it.
Updates #6995

Change-Id: I3355435db583755e0fc73d76347f6423b8939dfb
Signed-off-by: Brad Fitzpatrick <bradfitz@tailscale.com>
@bradfitz bradfitz merged commit 5ca22a0 into main Jan 20, 2023
@bradfitz bradfitz deleted the bradfitz/update_deb branch January 20, 2023 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants