Merge pull request #680 from snowflakedb/ignore405ForGetBucketAcceler…

…ateConfiguration

Ignore 405 error when S3 Acceleration is disabled on the S3 bucket

file_transfer_agent.go

@@ -615,6 +615,8 @@ func (sfa *snowflakeFileTransferAgent) transferAccelerateConfig() error {
 			if errors.As(err, &ae) {
 				if ae.ErrorCode() == "AccessDenied" {
 					return nil
+				} else if ae.ErrorCode() == "MethodNotAllowed" {
+					return nil
 				}
 			}
 			return err

file_transfer_agent_test.go

@@ -0,0 +1,44 @@
+// Copyright (c) 2022 Snowflake Computing Inc. All rights reserved.
+
+package gosnowflake
+
+import (
+	"context"
+	"errors"
+	"testing"
+
+	"github.com/aws/smithy-go"
+)
+
+func TestGetBucketAccelerateConfiguration(t *testing.T) {
+	if runningOnGithubAction() {
+		t.Skip("Should be run against an account in AWS EU North1 region.")
+	}
+	config, err := ParseDSN(dsn)
+	if err != nil {
+		t.Error(err)
+	}
+	sc, err := buildSnowflakeConn(context.Background(), *config)
+	if err != nil {
+		t.Fatal(err)
+	}
+	if err = authenticateWithConfig(sc); err != nil {
+		t.Fatal(err)
+	}
+	sfa := &snowflakeFileTransferAgent{
+		sc:          sc,
+		commandType: uploadCommand,
+		srcFiles:    make([]string, 0),
+		data: &execResponseData{
+			SrcLocations: make([]string, 0),
+		},
+	}
+	if err = sfa.transferAccelerateConfig(); err != nil {
+		var ae smithy.APIError
+		if errors.As(err, &ae) {
+			if ae.ErrorCode() == "MethodNotAllowed" {
+				t.Fatalf("should have ignored 405 error: %v", err)
+			}
+		}
+	}
+}