make client_scopes optional

cmu-sei · Oct 15, 2024 · 5df0155 · 5df0155
1 parent 816f65e
commit 5df0155
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/internal/provider/provider.go b/internal/provider/provider.go
@@ -89,9 +89,12 @@ func Provider() *schema.Provider {
 				Elem: &schema.Schema{
 					Type: schema.TypeString,
 				},
-				Required: true,
+				Optional: true,
 				DefaultFunc: func() (interface{}, error) {
-					return os.Getenv("SEI_CRUCIBLE_CLIENT_SCOPES"), nil
+					if val := os.Getenv("SEI_CRUCIBLE_CLIENT_SCOPES"); val != "" {
+						return val, nil
+					}
+					return []interface{}{}, nil // Return an empty list if the environment variable is not set
 				},
 			},
 		},

diff --git a/internal/util/util.go b/internal/util/util.go
@@ -26,6 +26,10 @@ func ToStringSlice(data *[]interface{}) *[]string {
 func GetAuth(m map[string]string) (string, error) {
 	scopes := strings.Split(m["client_scopes"], ",")
 
+	if len(scopes) == 0 || (len(scopes) == 1 && scopes[0] == "") {
+		scopes = nil
+	}
+
 	con := &oauth2.Config{
 		ClientID:     m["client_id"],
 		ClientSecret: m["client_secret"],