Fix version string and auth header (#201)

* fix: Strip newlines from version value

* fix: Set API key in headers not param

Author: brosenpod

api/query.go

@@ -8,6 +8,7 @@ import (
 	"net/http"
 	"os"
 	"runtime"
+	"strings"
 	"time"
 
 	"github.com/spf13/viper"
@@ -40,15 +41,17 @@ func Query(input Input) (res *http.Response, err error) {
 		return nil, errors.New("API key not found")
 	}
 
-	req, err := http.NewRequest("POST", apiUrl+"?api_key="+apiKey, bytes.NewBuffer(jsonValue))
+	req, err := http.NewRequest("POST", apiUrl, bytes.NewBuffer(jsonValue))
 	if err != nil {
 		return
 	}
 
-	userAgent := "RunPod-CLI/" + Version + " (" + runtime.GOOS + "; " + runtime.GOARCH + ")"
+	sanitizedVersion := strings.TrimRight(Version, "\r\n")
+	userAgent := "RunPod-CLI/" + sanitizedVersion + " (" + runtime.GOOS + "; " + runtime.GOARCH + ")"
 
 	req.Header.Add("Content-Type", "application/json")
 	req.Header.Set("User-Agent", userAgent)
+	req.Header.Set("Authorization", "Bearer "+apiKey)
 
 	client := &http.Client{Timeout: time.Second * 10}
 	return client.Do(req)

cmd/root.go

@@ -3,6 +3,7 @@ package cmd
 import (
 	"fmt"
 	"os"
+	"strings"
 
 	"github.com/runpod/runpodctl/api"
 	"github.com/runpod/runpodctl/cmd/config"
@@ -61,16 +62,21 @@ func registerCommands() {
 // Execute adds all child commands to the root command and sets flags appropriately.
 // This is called by main.main(). It only needs to happen once to the rootCmd.
 func Execute(ver string) {
-	version = ver
-	api.Version = ver
-	rootCmd.Version = ver
+	sanitizedVersion := sanitizeVersion(ver)
+	version = sanitizedVersion
+	api.Version = sanitizedVersion
+	rootCmd.Version = sanitizedVersion
 
 	if err := rootCmd.Execute(); err != nil {
 		fmt.Fprintf(os.Stderr, "Error: %v\n", err)
 		os.Exit(1)
 	}
 }
 
+func sanitizeVersion(ver string) string {
+	return strings.TrimRight(ver, "\r\n")
+}
+
 // initConfig reads in config file and ENV variables if set.
 func initConfig() {
 	home, err := os.UserHomeDir()