change from creation to exists check
All checks were successful
/ Make image (push) Successful in 1m14s
All checks were successful
/ Make image (push) Successful in 1m14s
This commit is contained in:
parent
38193e8943
commit
95b75d0067
3 changed files with 9 additions and 16 deletions
|
|
@ -32,7 +32,8 @@ func NewHandler(deps Deps) *Handler {
|
|||
"addr": media,
|
||||
}).Debug("Merch handler constructor | Media provider")
|
||||
|
||||
if err := media.СreateBucketIfNotExists(packageBucketName); err != nil {
|
||||
exists, err := media.CheckBucketExists(packageBucketName)
|
||||
if err != nil || !exists {
|
||||
log.WithError(err).Fatal("Merch handler constructor | Failed to ensure bucket exists")
|
||||
}
|
||||
|
||||
|
|
|
|||
|
|
@ -8,7 +8,7 @@ import (
|
|||
)
|
||||
|
||||
type MediaStorage interface {
|
||||
СreateBucketIfNotExists(bucketName string) error
|
||||
CheckBucketExists(bucketName string) (bool, error)
|
||||
Upload(ctx context.Context, bucket, object string, reader io.Reader, size int64) error
|
||||
Get(ctx context.Context, bucket, object string, expires time.Duration, params url.Values) (*url.URL, error)
|
||||
Delete(ctx context.Context, bucket, object string) error
|
||||
|
|
|
|||
|
|
@ -2,8 +2,6 @@ package mediaStorage
|
|||
|
||||
import (
|
||||
"context"
|
||||
"errors"
|
||||
"fmt"
|
||||
"github.com/minio/minio-go/v7"
|
||||
log "github.com/sirupsen/logrus"
|
||||
"io"
|
||||
|
|
@ -21,21 +19,15 @@ func newService(client *minio.Client) *Service {
|
|||
}
|
||||
}
|
||||
|
||||
func (s *Service) СreateBucketIfNotExists(bucketName string) error {
|
||||
func (s *Service) CheckBucketExists(bucketName string) (bool, error) {
|
||||
ctx := context.Background()
|
||||
err := s.client.MakeBucket(ctx, bucketName, minio.MakeBucketOptions{})
|
||||
exists, err := s.client.BucketExists(ctx, bucketName)
|
||||
if err != nil {
|
||||
var minioErr minio.ErrorResponse
|
||||
if errors.As(err, &minioErr) {
|
||||
if minioErr.Code == "BucketAlreadyExists" || minioErr.Code == "BucketAlreadyOwnedByYou" {
|
||||
log.Infof("Media storage | Bucket %s already exists, skipping creation", bucketName)
|
||||
return nil
|
||||
}
|
||||
}
|
||||
return fmt.Errorf("failed to create bucket: %w", err)
|
||||
log.WithError(err).Fatal("Media storage | Failed to check bucket existence")
|
||||
return exists, err
|
||||
}
|
||||
log.Infof("Media storage | Bucket %s created successfully", bucketName)
|
||||
return nil
|
||||
log.Infof("Media storage | Bucket %s exists", bucketName)
|
||||
return exists, nil
|
||||
}
|
||||
|
||||
func (s *Service) Upload(ctx context.Context, bucket, object string, reader io.Reader, size int64) error {
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue