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

Add no_proxy information to the README #56

Merged
merged 1 commit into from
Jan 23, 2024

Conversation

ccostino
Copy link
Contributor

This changeset adds a few extra details and instructions to the egress proxy configuration to address situations where developers have more than one application running within the platform.

Oftentimes, these applications will need to speak with each other (e.g., an API and a client), and there are secure and efficient ways to do that with the apps.internal domain routing via network policy configuration. However, with a proxy in front of the app, that traffic will not get through unless you explicitly configure your apps with the no_proxy environment variable and set it to include apps.internal as a domain to ignore.

This will allow internal traffic to flow as expected between your applications.

This changeset adds a few extra details and instructions to the egress proxy configuration to address situations where developers have more than one application running within the platform.

Oftentimes, these applications will need to speak with each other (e.g., an API and a client), and there are secure and efficient ways to do that with the `apps.internal` domain routing via network policy configuration.  However, with a proxy in front of the app, that traffic will not get through unless you explicitly configure your apps with the `no_proxy` environment variable and set it to include `apps.internal` as a domain to ignore.

This will allow internal traffic to flow as expected between your applications.
@ccostino ccostino added the documentation Improvements or additions to documentation label Jan 23, 2024
@ccostino ccostino requested a review from mogul January 23, 2024 16:53
@ccostino ccostino self-assigned this Jan 23, 2024
@mogul mogul merged commit 9cd9ae1 into main Jan 23, 2024
1 check passed
@mogul mogul deleted the ccostino/update-readme-with-no-proxy branch January 23, 2024 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants