feat: use PG* envvars

backend/cmd/scoreserver/config.go

@@ -7,23 +7,18 @@ import (
 
 	"github.com/cockroachdb/errors"
 	"github.com/ictsc/ictsc-regalia/backend/scoreserver"
+	"github.com/jackc/pgx/v5"
 )
 
 func newConfig() (*scoreserver.Config, error) {
-	var errs []error
-
 	adminHTTPAddr, err := netip.ParseAddrPort(*flagAdminHTTPAddr)
 	if err != nil {
-		errs = append(errs, errors.Wrap(err, "invalid admin HTTP address"))
-	}
-
-	dsn := os.Getenv("DB_DSN")
-	if dsn == "" {
-		errs = append(errs, errors.New("DB_DSN must be set"))
+		return nil, errors.Wrap(err, "invalid admin HTTP address")
 	}
 
-	if len(errs) > 0 {
-		return nil, errors.Join(errs...)
+	cfg, err := pgx.ParseConfig(os.Getenv("DB_DSN"))
+	if err != nil {
+		return nil, errors.Wrap(err, "failed to parse DB_DSN")
 	}
 
 	return &scoreserver.Config{
@@ -34,6 +29,6 @@ func newConfig() (*scoreserver.Config, error) {
 		ContestantHTTPAddress: netip.AddrPort{},
 		ContestantBaseURLs:    []url.URL{},
 
-		DBDSN: dsn,
+		PgConfig: *cfg,
 	}, nil
 }

backend/scoreserver/config.go

@@ -3,6 +3,8 @@ package scoreserver
 import (
 	"net/netip"
 	"net/url"
+
+	"github.com/jackc/pgx/v5"
 )
 
 type Config struct {
@@ -11,7 +13,7 @@ type Config struct {
 	ContestantHTTPAddress netip.AddrPort
 	ContestantBaseURLs    []url.URL
 
-	DBDSN string
+	PgConfig pgx.ConnConfig
 }
 
 type AdminAPIConfig struct {

backend/scoreserver/scoreserver.go

@@ -13,7 +13,6 @@ import (
 	"github.com/ictsc/ictsc-regalia/backend/pkg/pgxutil"
 	"github.com/ictsc/ictsc-regalia/backend/pkg/proto/admin/v1/adminv1connect"
 	"github.com/ictsc/ictsc-regalia/backend/scoreserver/admin"
-	"github.com/jackc/pgx/v5"
 	"github.com/jmoiron/sqlx"
 	"golang.org/x/net/http2"
 	"golang.org/x/net/http2/h2c"
@@ -31,12 +30,7 @@ type ScoreServer struct {
 }
 
 func New(cfg *Config) (*ScoreServer, error) {
-	pgcfg, err := pgx.ParseConfig(cfg.DBDSN)
-	if err != nil {
-		return nil, errors.Wrap(err, "failed to parse DB URL")
-	}
-
-	db := pgxutil.NewDBx(*pgcfg, pgxutil.WithOTel(true))
+	db := pgxutil.NewDBx(cfg.PgConfig, pgxutil.WithOTel(true))
 
 	adminServer := cfg.AdminAPI.new(db)
 

backend/scripts/local-exec

@@ -3,13 +3,13 @@
 
 compose_file="$(dirname "$0")/../compose.yaml"
 
-export DB_HOST=127.0.0.1
-DB_PORT="$(docker compose -f "$compose_file" port postgres 5432 | cut -d: -f2)"
-export DB_PORT
-export DB_USER=ictsc
+export PGHOST=127.0.0.1
+PGPORT="$(docker compose -f "$compose_file" port postgres 5432 | cut -d: -f2)"
+export PGPORT
+export PGUSER=ictsc
 export PGPASSWORD=password
-export DB_NAME=ictscore
-export DB_DSN="host=$DB_HOST port=$DB_PORT user=$DB_USER password=$PGPASSWORD dbname=$DB_NAME sslmode=disable"
+export PGDATABASE=ictscore
+export PGSSLMODE=disable
 
 export OTEL_METRICS_EXPORTER=${OTEL_METRICS_EXPORTER:-prometheus}
 

backend/scripts/migrate

@@ -2,10 +2,10 @@
 # vim: ft=sh :
 
 PSQLDEF="go run github.com/sqldef/sqldef/cmd/psqldef"
-if which psqldef > /dev/null; then
+if which psqldef 2>/dev/null; then
     PSQLDEF=psqldef
 fi
 
 schema_file=$(dirname "$0")/../schema.sql
 
-exec $PSQLDEF --host="$DB_HOST" --port="$DB_PORT" --user="$DB_USER" "$DB_NAME" < "$schema_file" "$@"
+exec $PSQLDEF --host="${PGHOST:-localhost}" --port="${PGPORT:-5432}" --user="${PGUSER:-ictsc}" "${PGDATABASE:-ictscore}" < "$schema_file" "$@"