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
fix: set proxy env variable when launching process #3838
Conversation
Fixes containers#3768 Signed-off-by: Jeff MAURY <jmaury@redhat.com>
Signed-off-by: Jeff MAURY <jmaury@redhat.com>
if (proxy.isEnabled()) { | ||
if (proxy.proxy?.httpsProxy) { | ||
env.HTTPS_PROXY = 'http://' + proxy.proxy.httpsProxy; | ||
} | ||
if (proxy.proxy?.httpProxy) { | ||
env.HTTP_PROXY = 'http://' + proxy.proxy.httpProxy; | ||
} | ||
if (proxy.proxy?.noProxy) { | ||
env.NO_PROXY = proxy.proxy.noProxy; | ||
} | ||
} | ||
|
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.
it means that if extension is setting a proxy data, it'll be replaced all the times by Podman Desktop ?
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.
Yes I don't see the use case
Signed-off-by: Jeff MAURY <jmaury@redhat.com>
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.
LGTM code wise
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.
LGTM
Fixes #3768
What does this PR do?
Set the proxy environment variables when launching process using the extension API
Screenshot/screencast of this PR
N/A
What issues does this PR fix or reference?
Fixes #3768
How to test this PR?