allow coder login for WSL #446
Conversation
Co-authored-by: Armin <[email protected]>
| @@ -66,7 +69,8 @@ func login(cmd *cobra.Command, workspaceURL *url.URL) error { | |||
| q.Add("show_token", "true") | |||
| authURL.RawQuery = q.Encode() | |||
|
|
|||
| if err := browser.OpenURL(authURL.String()); err != nil { | |||
| if err := openURL(authURL.String()); err != nil { | |||
| clog.LogWarn(err.Error()) | |||
There was a problem hiding this comment.
Thanks for the additional warning log 👍🏻
internal/cmd/login.go
Outdated
|
|
||
| // isWSL determines if coder-cli is running within Windows Subsystem for Linux | ||
| func isWSL() (bool, error) { | ||
| if runtime.GOOS == "darwin" || runtime.GOOS == "windows" { |
There was a problem hiding this comment.
it looks like we should make use of constants that are already defined:
we should add goosDarwin as a const as well
There was a problem hiding this comment.
Thank you for your review. Updated!
There was a problem hiding this comment.
This may resolve → #448
internal/cmd/login.go
Outdated
| if wsl { | ||
| cmd = "cmd.exe" | ||
| args = []string{"/c", "start"} | ||
| url = strings.Replace(url, "&", "^&", -1) |
There was a problem hiding this comment.
| url = strings.Replace(url, "&", "^&", -1) | |
| url = strings.ReplaceAll(url, "&", "^&") |
| var cmd string | ||
| var args []string | ||
|
|
||
| wsl, err := isWSL() |
There was a problem hiding this comment.
I think this is good, but I do wonder whether we could propose this upstream to pkg/browser
There was a problem hiding this comment.
Thank you for the review. I agree, this work should ideally be part of upstream pkg/browser. But Thoughts on moving forward with the work for Coder until I have the patch ready for upstream?
There was a problem hiding this comment.
Yeah, I think it makes sense to make this change now for sure! No objection to it :)
Thanks for your contribution!
Issue
coder loginfails to open user's browser while running from Windows Subsystem for Linux.Changes
coder loginto work while running from WSL.microsoftin/proc/versionfile.