add url encoding to provider for role names data source

Sean Sain · Sean Sain · commit 305d8d76cd64 · 2021-03-16T17:22:34.000-07:00
diff --git a/sumologic/data_source_sumologic_role.go b/sumologic/data_source_sumologic_role.go
@@ -5,6 +5,7 @@ import (
 	"errors"
 	"fmt"
 	"log"
+	"net/url"
 
 	"github.com/hashicorp/terraform-plugin-sdk/helper/schema"
 )
@@ -82,7 +83,7 @@ func dataSourceSumologicRoleRead(d *schema.ResourceData, meta interface{}) error
 }
 
 func (s *Client) GetRoleName(name string) (*Role, error) {
-	data, _, err := s.Get(fmt.Sprintf("v1/roles?name=%s", name))
+	data, _, err := s.Get(fmt.Sprintf("v1/roles?name=%s", url.QueryEscape(name)))
 	if err != nil {
 		return nil, err
 	}

Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,7 @@ import (`
`5`	`5`	`"errors"`
`6`	`6`	`"fmt"`
`7`	`7`	`"log"`
	`8`	`+ "net/url"`
`8`	`9`
`9`	`10`	`"github.com/hashicorp/terraform-plugin-sdk/helper/schema"`
`10`	`11`	`)`
`@@ -82,7 +83,7 @@ func dataSourceSumologicRoleRead(d *schema.ResourceData, meta interface{}) error`
`82`	`83`	`}`
`83`	`84`
`84`	`85`	`func (s Client) GetRoleName(name string) (Role, error) {`
`85`		`- data, _, err := s.Get(fmt.Sprintf("v1/roles?name=%s", name))`
	`86`	`+ data, _, err := s.Get(fmt.Sprintf("v1/roles?name=%s", url.QueryEscape(name)))`
`86`	`87`	`if err != nil {`
`87`	`88`	`return nil, err`
`88`	`89`	`}`