test

neicnordic · Aug 31, 2023 · 088e9c9 · 088e9c9
1 parent 289f2b2
commit 088e9c9
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/sda/cmd/s3inbox/healthchecks.go b/sda/cmd/s3inbox/healthchecks.go
@@ -9,7 +9,6 @@ import (
 	"time"
 
 	"github.com/neicnordic/sensitive-data-archive/internal/config"
-
 	"github.com/heptiolabs/healthcheck"
 )
 
@@ -26,8 +25,11 @@ type HealthCheck struct {
 // the backend S3 storage and the Message Broker so it can report readiness.
 func NewHealthCheck(port int, db *sql.DB, conf *config.Config, tlsConfig *tls.Config) *HealthCheck {
 	s3URL := conf.Inbox.S3.URL
+	if conf.Inbox.S3.Port != 0 {
+		s3URL = fmt.Sprintf("%s:%d", s3URL, conf.Inbox.S3.Port)
+	}
 	if conf.Inbox.S3.Readypath != "" {
-		s3URL = conf.Inbox.S3.URL + conf.Inbox.S3.Readypath
+		s3URL = s3URL + conf.Inbox.S3.Readypath
 	}
 
 	brokerURL := fmt.Sprintf("%s:%d", conf.Broker.Host, conf.Broker.Port)

diff --git a/sda/internal/config/config.go b/sda/internal/config/config.go
@@ -669,7 +669,6 @@ func configS3Storage(prefix string) storage.S3Conf {
 
 	// Defaults (move to viper?)
 
-	s3.Port = 443
 	s3.Region = "us-east-1"
 	s3.NonExistRetryTime = 2 * time.Minute