Skip to content

Cf neworking tls update #212

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

Merged
merged 4 commits into from
Nov 1, 2021
Merged

Cf neworking tls update #212

merged 4 commits into from
Nov 1, 2021

Conversation

MarcPaquette
Copy link

@MarcPaquette MarcPaquette commented Oct 28, 2021

The diagrams in "Securing Traffic" section are incorrect. They show traffic being unencrypted between gorouter + diego cells. Additionally some sections make reference to unencrypted app traffic for windows, and some don't. Some don't make explicit reference to linux app traffic being encrypted between router + diego.

These changes should apply to all supported versions for the docs.

The diagrams are incorrect. They show traffic being unencrypted between gorouter + diego cells. Additionally some sections make reference to unencrypted app traffic for windows, and some don't. Some don't make explicit reference to linux app traffic being encrypted between router + diego.

These changes should apply to all supported docs
@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.

@mjgutermuth
Copy link
Contributor

Thanks @MarcPaquette!

@mjgutermuth mjgutermuth deleted the cf-neworking-tls-update branch November 1, 2021 21:09
mjgutermuth added a commit that referenced this pull request Nov 2, 2021
mjgutermuth added a commit that referenced this pull request Nov 2, 2021
mjgutermuth added a commit that referenced this pull request Nov 2, 2021
mjgutermuth added a commit that referenced this pull request Nov 2, 2021
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.

4 participants