-
Notifications
You must be signed in to change notification settings - Fork 7.7k
update container proxy behaviour with PAC file #22847
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
base: main
Are you sure you want to change the base?
update container proxy behaviour with PAC file #22847
Conversation
✅ Deploy Preview for docsdocker ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
@Xeeynamo can you also take a look please and add/modify if needed ? Thank you. |
This should be merged after release of DD 4.42.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
@@ -52,7 +52,7 @@ The `containersProxy` setting describes the policy which is applied to traffic f | |||
> [!IMPORTANT] | |||
> | |||
> Any existing `proxy` setting in the `admin-settings.json` file continues to apply to traffic from the app on the host. | |||
> If the PAC file download fails, the Docker Desktop app and its containers do not block the request; instead, they attempt to connect directly to the target URL. | |||
> If the PAC file download fails, the containers block the request to the target URL. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we need to specify that this is only with version 4.42.1 and later?
Description
Related issues or tickets
Relates to this code changes in https://github.com/docker/pinata/pull/34220 and https://github.com/docker/pinata/pull/34955. This PR updates the docs to reflect the new behaviour.
Reviews