From 8c7a07c0cb705c4447538cde2ad7268277c74a0c Mon Sep 17 00:00:00 2001 From: Shivang Yadav <125182653+shivang-16@users.noreply.github.com> Date: Sat, 29 Aug 2026 12:48:10 +0530 Subject: [PATCH 1/3] Simplify Open function by removing error check Removed error handling for database connection opening. --- internal/infra/postgres/postgres.go | 7 +------ 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/internal/infra/postgres/postgres.go b/internal/infra/postgres/postgres.go index 27072b0..e8f2895 100644 --- a/internal/infra/postgres/postgres.go +++ b/internal/infra/postgres/postgres.go @@ -17,18 +17,13 @@ type Client struct { func Open(cfg config.Postgres) (*Client, error) { db, err := sql.Open("pgx", cfg.DSN()) - if err != nil { - return nil, fmt.Errorf("open postgres: %w", err) - } maxOpen := cfg.MaxOpenConns if maxOpen < 1 { maxOpen = 10 } maxIdle := cfg.MaxIdleConns - if maxIdle < 1 { - maxIdle = 5 - } + db.SetMaxOpenConns(maxOpen) db.SetMaxIdleConns(maxIdle) db.SetConnMaxLifetime(time.Hour) From 8c035f6416f467cb312a5a94811e900624d50020 Mon Sep 17 00:00:00 2001 From: Shivang Yadav <125182653+shivang-16@users.noreply.github.com> Date: Sat, 29 Aug 2026 12:52:58 +0530 Subject: [PATCH 2/3] Update postgres.go --- internal/infra/postgres/postgres.go | 5 +---- 1 file changed, 1 insertion(+), 4 deletions(-) diff --git a/internal/infra/postgres/postgres.go b/internal/infra/postgres/postgres.go index e8f2895..97f6082 100644 --- a/internal/infra/postgres/postgres.go +++ b/internal/infra/postgres/postgres.go @@ -31,10 +31,7 @@ func Open(cfg config.Postgres) (*Client, error) { ctx, cancel := context.WithTimeout(context.Background(), 5*time.Second) defer cancel() - if err := db.PingContext(ctx); err != nil { - _ = db.Close() - return nil, fmt.Errorf("ping postgres: %w", err) - } + return &Client{db: db}, nil } From 456c3f91fbef344b556158eb49edee9a9249b636 Mon Sep 17 00:00:00 2001 From: Shivang Yadav <125182653+shivang-16@users.noreply.github.com> Date: Sat, 29 Aug 2026 13:19:42 +0530 Subject: [PATCH 3/3] Update postgres.go --- internal/infra/postgres/postgres.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/internal/infra/postgres/postgres.go b/internal/infra/postgres/postgres.go index 97f6082..d511525 100644 --- a/internal/infra/postgres/postgres.go +++ b/internal/infra/postgres/postgres.go @@ -38,9 +38,6 @@ func Open(cfg config.Postgres) (*Client, error) { func OpenAndMigrate(cfg config.Postgres, migrationsDir string) (*Client, error) { client, err := Open(cfg) - if err != nil { - return nil, err - } ctx, cancel := context.WithTimeout(context.Background(), 10*time.Second) defer cancel()