Merge pull request #22 from CarlQLange/new-with-region

ctrox · web-flow · commit 6a23dbae7249 · 2019-07-26T02:56:09.000+02:00
Use NewWithRegion not minio's region resolution
diff --git a/pkg/s3/s3-client.go b/pkg/s3/s3-client.go
@@ -41,7 +41,7 @@ func newS3Client(cfg *Config) (*s3Client, error) {
 	if u.Port() != "" {
 		endpoint = u.Hostname() + ":" + u.Port()
 	}
-	minioClient, err := minio.New(endpoint, client.cfg.AccessKeyID, client.cfg.SecretAccessKey, ssl)
+	minioClient, err := minio.NewWithRegion(endpoint, client.cfg.AccessKeyID, client.cfg.SecretAccessKey, ssl, client.cfg.Region)
 	if err != nil {
 		return nil, err
 	}

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ func newS3Client(cfg Config) (s3Client, error) {`
`41`	`41`	`if u.Port() != "" {`
`42`	`42`	`endpoint = u.Hostname() + ":" + u.Port()`
`43`	`43`	`}`
`44`		`- minioClient, err := minio.New(endpoint, client.cfg.AccessKeyID, client.cfg.SecretAccessKey, ssl)`
	`44`	`+ minioClient, err := minio.NewWithRegion(endpoint, client.cfg.AccessKeyID, client.cfg.SecretAccessKey, ssl, client.cfg.Region)`
`45`	`45`	`if err != nil {`
`46`	`46`	`return nil, err`
`47`	`47`	`}`