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

Manage Firewall rules for DB Direct allows IPs #15610

Merged
merged 24 commits into from
Apr 27, 2020

Conversation

jgoizueta
Copy link
Member

@jgoizueta jgoizueta commented Apr 21, 2020

We need to create and delete Firewall rules as appropriate to allow the IPs enabled by each organizations for DB direct to reach pgproxy servers.

Firewall rules won't be managed unless it is enabled through the config parameter dbdirect.firewall.enabled.

When enabled a single rule is maintained for each organization; the name of the rule is defined by DbdirectIp#firewall_rule_name and uses a template configured by dbdirect.firewall.rule_name.

Credentials for using the Google Cloud API to configure the firewall are not handled here (in production we'll be using host service accounts for this purpose)

@jgoizueta jgoizueta changed the base branch from dbdirect to master April 23, 2020 11:00
Copy link
Contributor

@gonzaloriestra gonzaloriestra left a comment

Choose a reason for hiding this comment

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

👏

app/models/carto/dbdirect_ip.rb Outdated Show resolved Hide resolved
lib/carto/dbdirect/firewall_manager.rb Outdated Show resolved Hide resolved
jgoizueta and others added 6 commits April 27, 2020 11:55
Co-Authored-By: Gonzalo Riestra <gonzalor@cartodb.com>
…github.com:CartoDB/cartodb into jgoizueta/ch66432/db-direct-allowed-ips-management
@jgoizueta jgoizueta merged commit 53ced16 into master Apr 27, 2020
@jgoizueta jgoizueta deleted the jgoizueta/ch66432/db-direct-allowed-ips-management branch April 27, 2020 11:18
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.

None yet

2 participants