update routing_rules to TypeList

Author: anGie44

internal/service/s3/bucket_website_configuration.go

@@ -89,7 +89,7 @@ func ResourceBucketWebsiteConfiguration() *schema.Resource {
 				},
 			},
 			"routing_rule": {
-				Type:     schema.TypeSet,
+				Type:     schema.TypeList,
 				Optional: true,
 				Elem: &schema.Resource{
 					Schema: map[string]*schema.Schema{
@@ -167,8 +167,8 @@ func resourceBucketWebsiteConfigurationCreate(ctx context.Context, d *schema.Res
 		websiteConfig.RedirectAllRequestsTo = expandS3BucketWebsiteConfigurationRedirectAllRequestsTo(v.([]interface{}))
 	}
 
-	if v, ok := d.GetOk("routing_rule"); ok && v.(*schema.Set).Len() > 0 {
-		websiteConfig.RoutingRules = expandS3BucketWebsiteConfigurationRoutingRules(v.(*schema.Set).List())
+	if v, ok := d.GetOk("routing_rule"); ok && len(v.([]interface{})) > 0 && v.([]interface{})[0] != nil {
+		websiteConfig.RoutingRules = expandS3BucketWebsiteConfigurationRoutingRules(v.([]interface{}))
 	}
 
 	input := &s3.PutBucketWebsiteInput{
@@ -270,8 +270,8 @@ func resourceBucketWebsiteConfigurationUpdate(ctx context.Context, d *schema.Res
 		websiteConfig.RedirectAllRequestsTo = expandS3BucketWebsiteConfigurationRedirectAllRequestsTo(v.([]interface{}))
 	}
 
-	if v, ok := d.GetOk("routing_rule"); ok && v.(*schema.Set).Len() > 0 {
-		websiteConfig.RoutingRules = expandS3BucketWebsiteConfigurationRoutingRules(v.(*schema.Set).List())
+	if v, ok := d.GetOk("routing_rule"); ok && len(v.([]interface{})) > 0 && v.([]interface{})[0] != nil {
+		websiteConfig.RoutingRules = expandS3BucketWebsiteConfigurationRoutingRules(v.([]interface{}))
 	}
 
 	input := &s3.PutBucketWebsiteInput{

website/docs/r/s3_bucket_website_configuration.html.markdown

@@ -44,7 +44,7 @@ The following arguments are supported:
 * `expected_bucket_owner` - (Optional, Forces new resource) The account ID of the expected bucket owner.
 * `index_document` - (Optional, Required if `redirect_all_requests_to` is not specified) The name of the index document for the website [detailed below](#index_document).
 * `redirect_all_requests_to` - (Optional, Required if `index_document` is not specified) The redirect behavior for every request to this bucket's website endpoint [detailed below](#redirect_all_requests_to). Conflicts with `error_document`, `index_document`, and `routing_rule`.
-* `routing_rule` - (Optional, Conflicts with `redirect_all_requests_to`) Set of rules that define when a redirect is applied and the redirect behavior [detailed below](#routing_rule).
+* `routing_rule` - (Optional, Conflicts with `redirect_all_requests_to`) List of rules that define when a redirect is applied and the redirect behavior [detailed below](#routing_rule).
 
 ### error_document