Added token validation for login command

cmd/login.go

@@ -9,11 +9,13 @@ import (
 
 	"github.com/spf13/cobra"
 
+	"github.com/opf/openproject-cli/components/common"
 	"github.com/opf/openproject-cli/components/configuration"
 	"github.com/opf/openproject-cli/components/parser"
 	"github.com/opf/openproject-cli/components/paths"
 	"github.com/opf/openproject-cli/components/printer"
 	"github.com/opf/openproject-cli/components/requests"
+	"github.com/opf/openproject-cli/components/resources/users"
 	"github.com/opf/openproject-cli/dtos"
 )
 
@@ -65,7 +67,20 @@ func login(_ *cobra.Command, _ []string) {
 			continue
 		}
 
-		token = t
+		token = common.SanitizeLineBreaks(t)
+
+		requests.Init(hostUrl, token)
+		user, err := users.Me()
+		if err != nil {
+			printer.Error(err)
+			continue
+		}
+
+		if user.Name == "Anonymous" {
+			printer.ErrorText("no authenticate given")
+			continue
+		}
+
 		break
 	}
 

components/common/string.go

@@ -0,0 +1,7 @@
+package common
+
+import "strings"
+
+func SanitizeLineBreaks(input string) string {
+	return strings.Replace(input, "\n", "", -1)
+}

components/common/string_test.go

@@ -0,0 +1,18 @@
+package common_test
+
+import (
+	"testing"
+
+	"github.com/opf/openproject-cli/components/common"
+)
+
+func TestSanitizeLineBreaks(t *testing.T) {
+	input := "abc\n"
+	expected := "abc"
+
+	result := common.SanitizeLineBreaks(input)
+
+	if result != expected {
+		t.Errorf("Expected %s, but got %s", expected, result)
+	}
+}

components/configuration/util.go

@@ -6,6 +6,7 @@ import (
 	"path/filepath"
 	"strings"
 
+	"github.com/opf/openproject-cli/components/common"
 	"github.com/opf/openproject-cli/components/errors"
 )
 
@@ -44,8 +45,7 @@ func ReadConfig() (host, token string, err error) {
 		return "", "", nil
 	}
 
-	sanitized := strings.Replace(string(file), "\n", "", -1)
-	parts := strings.Split(sanitized, " ")
+	parts := strings.Split(common.SanitizeLineBreaks(string(file)), " ")
 	if len(parts) != 2 {
 		return "", "", errors.Custom(fmt.Sprintf("Invalid config file at %s. Please remove the file and run `op login` again.", configFile()))
 	}

components/paths/paths.go

@@ -34,6 +34,10 @@ func Users() string {
 	return Root() + "/users"
 }
 
+func UserMe() string {
+	return Users() + "/me"
+}
+
 func WorkPackage(id uint64) string {
 	return WorkPackages() + fmt.Sprintf("/%d", id)
 }

components/printer/common.go

@@ -62,7 +62,7 @@ func apiError(status int, err apiErrorModel) {
 		status,
 		Yellow(err.ErrorIdentifier),
 		err.Messsage,
-		)
+	)
 }
 
 func indent(spaces int) (res string) {

components/requests/requests.go

@@ -38,7 +38,7 @@ func Patch(path string, requestBody *RequestData) (responseBody []byte, err erro
 
 func Do(method string, path string, query *Query, requestData *RequestData) (responseBody []byte, err error) {
 	if client == nil || hostUnitialised() {
-		return  nil, errors.Custom("Cannot execute requests without initializing request client first. Run `op login`")
+		return nil, errors.Custom("Cannot execute requests without initializing request client first. Run `op login`")
 	}
 
 	requestUrl := *host
@@ -58,7 +58,7 @@ func Do(method string, path string, query *Query, requestData *RequestData) (res
 		body,
 	)
 	if err != nil {
-		return  nil, err
+		return nil, err
 	}
 
 	if requestData != nil {
@@ -71,14 +71,13 @@ func Do(method string, path string, query *Query, requestData *RequestData) (res
 
 	resp, err := client.Do(request)
 	if err != nil {
-		return  nil, err
+		return nil, err
 	}
 
-
 	defer func() { _ = resp.Body.Close() }()
 	response, err := io.ReadAll(resp.Body)
 	if err != nil {
-		return  nil, err
+		return nil, err
 	}
 
 	if !isSuccess(resp.StatusCode) {

components/resources/users/functions.go

@@ -2,6 +2,7 @@ package users
 
 import (
 	"github.com/opf/openproject-cli/components/parser"
+	"github.com/opf/openproject-cli/components/paths"
 	"github.com/opf/openproject-cli/components/printer"
 	"github.com/opf/openproject-cli/components/requests"
 	"github.com/opf/openproject-cli/dtos"
@@ -30,3 +31,13 @@ func ByIds(ids []uint64) []*models.User {
 	userCollection := parser.Parse[dtos.UserCollectionDto](response)
 	return userCollection.Convert()
 }
+
+func Me() (*models.User, error) {
+	response, err := requests.Get(paths.UserMe(), nil)
+	if err != nil {
+		return nil, err
+	}
+
+	user := parser.Parse[dtos.UserDto](response)
+	return user.Convert(), nil
+}