Remove workaround for Konnect API bug which has been fixed

Author: scastrianni

docs/data-sources/service.md

@@ -19,7 +19,7 @@ data "konnect_service" "example" {
 * `name` - **(Optional, String)** The filter string to apply to the name of the service. Uses equality.
 ## Attribute Reference
 * `retries` - **(Integer)** The number of retries to execute upon failure to proxy.
-* `protocol` - **(Integer)** The protocol used to communicate with the host.
+* `protocol` - **(String)** The protocol used to communicate with the host.
 * `port` - **(Integer)** The port used to communicate with the host.
 * `path` - **(String)** The path to be used in requests to the host.
 * `connect_timeout` - **(Integer)** The timeout in milliseconds for establishing a connection to the host.

konnect/client/service.go

@@ -27,12 +27,11 @@ type ServiceCollection struct {
 	Services []Service `json:"data"`
 }
 type ListService struct {
-	ControlPlaneId string `json:"-"`
-	Id             string `json:"id"`
-	Name           string `json:"name"`
-	Retries        int    `json:"retries"`
-	//This is a bug in Konnect API as protocol should be a string
-	Protocol       int      `json:"protocol"`
+	ControlPlaneId string   `json:"-"`
+	Id             string   `json:"id"`
+	Name           string   `json:"name"`
+	Retries        int      `json:"retries"`
+	Protocol       string   `json:"protocol"`
 	Host           string   `json:"host"`
 	Port           int      `json:"port"`
 	Path           string   `json:"path"`

konnect/data_source_service.go

@@ -39,7 +39,7 @@ func dataSourceService() *schema.Resource {
 				Computed: true,
 			},
 			"protocol": {
-				Type:     schema.TypeInt,
+				Type:     schema.TypeString,
 				Computed: true,
 			},
 			"port": {