Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement Address Whitelist Functionality #813

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

AndrewJackson2020
Copy link
Contributor

@AndrewJackson2020 AndrewJackson2020 commented Sep 10, 2024

This PR adds address whitelist functionality to pgcat. This is mostly useful with trust auth where no password is required but you can still allow only specific IP's/IP ranges/addresses to access a user/database but can also be used with password authentication to provide an extra layer of security.

This PR is unit tested, does not require any changes to the dockerfile/CI. It does require a slight change in dependencies of the rust cargo dependencies but I don't think this should break anything.

One potential issue with this implementation is that I had to make the set element of the AddrSet struct public. I am open to any alternative implementations that do this in a more clean way.

@AndrewJackson2020 AndrewJackson2020 changed the title add ip whitelist functionality Address Whitelist Functionality Sep 10, 2024
@AndrewJackson2020 AndrewJackson2020 changed the title Address Whitelist Functionality Implement Address Whitelist Functionality Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant