Skip to content
This repository has been archived by the owner on Feb 11, 2020. It is now read-only.

Stop allowing the entire internet access to the db #354

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

matthewfischer
Copy link
Contributor

As a part of working with the doc team on the GCP setup instructions, we
are recommending that this setting is removed. All the PCF components
can communicate with Cloud SQL via the NATs and therefore this is not
necessary and is insecure.

Thanks for submitting an pull request to pcf-pipelines.

To speed up the process of reviewing your pull request please provide us with:

  • A short explanation of the proposed change:

  • An explanation of the use cases your change solves:

  • Expected result after the change:

  • Current result before the change:

  • Links to any other associated PRs or issues:

  • I have viewed signed and have submitted the Contributor License Agreement

  • I have made this pull request to the master branch

  • I have run all the unit tests

As a part of working with the doc team on the GCP setup instructions, we
are recommending that this setting is removed. All the PCF components
can communicate with Cloud SQL via the NATs and therefore this is not
necessary and is insecure.
@cf-gitbot
Copy link

We have created an issue in Pivotal Tracker to manage this. Unfortunately, the Pivotal Tracker project is private so you may be unable to view the contents of the story.

The labels on this github issue will be updated when the story is started.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants