Use different creds than the default ones

Signed-off-by: Gabriel Adrian Samfira <[email protected]>

Author: gabriel-samfira

cmd/garm-cli/cmd/github_credentials.go

@@ -15,6 +15,8 @@
 package cmd
 
 import (
+	"crypto/x509"
+	"encoding/pem"
 	"fmt"
 	"os"
 	"strconv"
@@ -41,7 +43,7 @@ var (
 var credentialsCmd = &cobra.Command{
 	Use:     "credentials",
 	Aliases: []string{"creds"},
-	Short:   "List configured credentials",
+	Short:   "List configured credentials. This is an alias for the github credentials command.",
 	Long: `List all available github credentials.
 
 This command is an alias for the garm-cli github credentials command.`,
@@ -258,20 +260,20 @@ func init() {
 }
 
 func parsePrivateKeyFromPath(path string) ([]byte, error) {
-	// if _, err := os.Stat(path); err != nil {
-	// 	return nil, fmt.Errorf("private key file not found: %s", credentialsPrivateKeyPath)
-	// }
+	if _, err := os.Stat(path); err != nil {
+		return nil, fmt.Errorf("private key file not found: %s", credentialsPrivateKeyPath)
+	}
 	keyContents, err := os.ReadFile(path)
 	if err != nil {
 		return nil, fmt.Errorf("failed to read private key file: %w", err)
 	}
-	// pemBlock, _ := pem.Decode(keyContents)
-	// if pemBlock == nil {
-	// 	return nil, fmt.Errorf("failed to decode PEM block")
-	// }
-	// if _, err := x509.ParsePKCS1PrivateKey(pemBlock.Bytes); err != nil {
-	// 	return nil, fmt.Errorf("failed to parse private key: %w", err)
-	// }
+	pemBlock, _ := pem.Decode(keyContents)
+	if pemBlock == nil {
+		return nil, fmt.Errorf("failed to decode PEM block")
+	}
+	if _, err := x509.ParsePKCS1PrivateKey(pemBlock.Bytes); err != nil {
+		return nil, fmt.Errorf("failed to parse private key: %w", err)
+	}
 	return keyContents, nil
 }
 

params/params.go

@@ -585,6 +585,7 @@ type GithubCredentials struct {
 	Enterprises   []Enterprise   `json:"enterprises,omitempty"`
 	Endpoint      GithubEndpoint `json:"endpoint"`
 
+	// Do not serialize sensitive info.
 	CredentialsPayload []byte `json:"-"`
 }
 

test/integration/e2e/credentials.go

@@ -62,8 +62,8 @@ func TestGithubCredentialsFailsToDeleteWhenInUse() {
 
 	repo := CreateRepo("dummy-owner", "dummy-repo", creds.Name, "superSecret@123BlaBla")
 	defer func() {
-		deleteRepo(cli, authToken, repo.ID)
 		deleteGithubCredentials(cli, authToken, int64(creds.ID))
+		deleteRepo(cli, authToken, repo.ID)
 	}()
 
 	if err := deleteGithubCredentials(cli, authToken, int64(creds.ID)); err == nil {
@@ -170,7 +170,7 @@ func TestGithubCredentialsFailWhenEndpointDoesntExist() {
 	slog.Info("Testing error when endpoint doesn't exist")
 	createCredsParams := params.CreateGithubCredentialsParams{
 		Name:        dummyCredentialsName,
-		Endpoint:    defaultEndpointName,
+		Endpoint:    "iDontExist.example.com",
 		Description: "GARM test credentials",
 		AuthType:    params.GithubAuthTypePAT,
 		PAT: params.GithubPAT{