Print error when --project-id flag is not set for gce and packet

Prints an error message prompting to set the --project-id flag
when using gce or packet as the provider
Fixes #52

Signed-off-by: Utsav Anand <utsavanand2@gmail.com>

cmd/create.go

@@ -6,11 +6,12 @@ package cmd
 import (
 	"encoding/base64"
 	"fmt"
-	"github.com/inlets/inletsctl/pkg/env"
 	"strconv"
 	"strings"
 	"time"
 
+	"github.com/inlets/inletsctl/pkg/env"
+
 	"github.com/inlets/inletsctl/pkg/names"
 	"github.com/inlets/inletsctl/pkg/provision"
 
@@ -126,24 +127,26 @@ func runCreate(cmd *cobra.Command, _ []string) error {
 
 	var secretKey string
 	var organisationID string
+	var projectID string
 	if provider == "scaleway" || provider == "ec2" {
 
 		var secretKeyErr error
-		secretKey, secretKeyErr = env.GetRequiredFileOrString(cmd.Flags(),
-			"secret-key-file",
-			"secret-key",
-			"INLETS_SECRET_KEY",
-		)
+		secretKey, secretKeyErr = getFileOrString(cmd.Flags(), "secret-key-file", "secret-key", true)
 		if secretKeyErr != nil {
 			return secretKeyErr
 		}
 
 		if provider == "scaleway" {
 			organisationID, _ = cmd.Flags().GetString("organisation-id")
 			if len(organisationID) == 0 {
-				return fmt.Errorf("--organisation-id cannot be empty")
+				return fmt.Errorf("--organisation-id flag must be set")
 			}
 		}
+	} else if provider == "gce" || provider == "packet" {
+		projectID, _ = cmd.Flags().GetString("project-id")
+		if len(projectID) == 0 {
+			return fmt.Errorf("--project-id flag must be set")
+		}
 	}
 
 	provisioner, err := getProvisioner(provider, accessToken, secretKey, organisationID, region)
@@ -162,8 +165,6 @@ func runCreate(cmd *cobra.Command, _ []string) error {
 
 	userData := makeUserdata(inletsToken, inletsControlPort, remoteTCP)
 
-	projectID, _ := cmd.Flags().GetString("project-id")
-
 	hostReq, err := createHost(provider, name, region, zone, projectID, userData, strconv.Itoa(inletsControlPort), pro)
 	if err != nil {
 		return err