Fix coder login on Windows (#310)

* switch to scanner for token

* fix spacing

* ignore return value from scanner.Scan()

Author: bpmct

internal/cmd/login.go

@@ -4,7 +4,6 @@ import (
 	"bufio"
 	"context"
 	"fmt"
-	"io"
 	"net/url"
 	"strings"
 
@@ -73,7 +72,15 @@ func login(cmd *cobra.Command, envURL *url.URL) error {
 		fmt.Printf("Your browser has been opened to visit:\n\n\t%s\n\n", authURL.String())
 	}
 
-	token := readLine("Paste token here: ", cmd.InOrStdin())
+	fmt.Print("Paste token here: ")
+	var token string
+	scanner := bufio.NewScanner(cmd.InOrStdin())
+	_ = scanner.Scan()
+	token = scanner.Text()
+	if err := scanner.Err(); err != nil {
+		return xerrors.Errorf("reading standard input: %w", err)
+	}
+
 	if err := pingAPI(cmd.Context(), envURL, token); err != nil {
 		return xerrors.Errorf("ping API with credentials: %w", err)
 	}
@@ -84,13 +91,6 @@ func login(cmd *cobra.Command, envURL *url.URL) error {
 	return nil
 }
 
-func readLine(prompt string, r io.Reader) string {
-	reader := bufio.NewReader(r)
-	fmt.Print(prompt)
-	text, _ := reader.ReadString('\n')
-	return strings.TrimSuffix(text, "\n")
-}
-
 // pingAPI creates a client from the given url/token and try to exec an api call.
 // Not using the SDK as we want to verify the url/token pair before storing the config files.
 func pingAPI(ctx context.Context, envURL *url.URL, token string) error {