Skip to content
This repository was archived by the owner on Apr 30, 2026. It is now read-only.

feat: upgrade to postgres 17#90

Merged
mishraomp merged 13 commits into
mainfrom
feat/pg-17
Jun 13, 2025
Merged

feat: upgrade to postgres 17#90
mishraomp merged 13 commits into
mainfrom
feat/pg-17

Conversation

@mishraomp
Copy link
Copy Markdown
Collaborator

No description provided.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR upgrades the project’s PostgreSQL stack to version 17 and adds SSL requirements to relevant services.

  • Bumps Aurora PostgreSQL engine and parameter group families to 17
  • Adds SSL mode settings to ECS task definitions and Prisma data source
  • Updates Docker image tag and README to match Postgres 17

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
infrastructure/frontend/outputs.tf Added a url output for the CloudFront distribution
infrastructure/database/aurora-v2.tf Updated RDS engine version and parameter group family
infrastructure/api/ecs.tf Appended ?sslmode=require to Flyway JDBC URL
docker-compose.yml Switched PostGIS image tag to 17-3.5
backend/src/prisma.service.ts Introduced sslmode parameter in Prisma connection URL
README.md Updated Postgres version requirement to 17.4
Comments suppressed due to low confidence (3)

docker-compose.yml:19

  • The comment indicates PostGIS 3.4 but the image is tagged 3.5; update the comment to reflect the actual PostGIS version.
image: postgis/postgis:17-3.5 # Updated to PostgreSQL 17 with PostGIS 3.4

backend/src/prisma.service.ts:11

  • [nitpick] The inline comment refers to AWS deployments generally requiring SSL; consider clarifying that this is specific to your Aurora configuration to avoid confusion.
const SSL_MODE = process.env.NODE_ENV === 'local' ? 'prefer' : 'require'; // 'require' for aws deployments, 'prefer' for local development

infrastructure/frontend/outputs.tf:6

  • [nitpick] Align with existing HCL formatting: add a space around = and match the indentation of other map entries (e.g., url = ...).
url= "https://${aws_cloudfront_distribution.s3_distribution.domain_name}"

Comment thread .github/workflows/.builds.yml Fixed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

upgrade aurora postgres to 17

4 participants