Skip to content

Conversation

ZdravkoDonev-redis
Copy link
Collaborator

Currently we've documented that you should use RDS Proxy for all source databases, but that is not the case for the RDS Proxy. We have an initiative to try and improve this, but for the time being we must document the workaround for that database.
So here it is.

Copy link

jit-ci bot commented Oct 2, 2025

Hi, I’m Jit, a friendly security platform designed to help developers build secure applications from day zero with an MVS (Minimal viable security) mindset.

In case there are security findings, they will be communicated to you as a comment inside the PR.

Hope you’ll enjoy using Jit.

Questions? Comments? Want to learn more? Get in touch with us.

Copy link
Contributor

github-actions bot commented Oct 2, 2025

Copy link
Contributor

@Copilot 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 adds documentation clarifying that RDS Proxy is not compatible with PostgreSQL databases and provides alternative setup instructions. The changes address a gap in the current documentation which incorrectly suggested using RDS Proxy for all source databases.

  • Added warning about RDS Proxy incompatibility with PostgreSQL databases
  • Provided alternative approaches for PostgreSQL in test and production environments
  • Updated setup instructions to differentiate between MySQL/SQL Server and PostgreSQL workflows

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@cmilesb cmilesb self-assigned this Oct 2, 2025
@cmilesb cmilesb requested a review from a team October 2, 2025 17:00
@cmilesb cmilesb merged commit 2ea4141 into preview-rc-rdi Oct 3, 2025
6 checks passed
@cmilesb cmilesb deleted the RDSC-4040-add-note-for-RDS-proxy branch October 3, 2025 18:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants