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

no_proxy is not respected during index searches #5497

Closed
minsis opened this issue Dec 9, 2021 · 2 comments · Fixed by #5500
Closed

no_proxy is not respected during index searches #5497

minsis opened this issue Dec 9, 2021 · 2 comments · Fixed by #5500

Comments

@minsis
Copy link
Contributor

minsis commented Dec 9, 2021

SUMMARY

This isn't exactly a bug (kinda?) but maybe a solution to consider in how implement this.

Basically no_proxy is ignored during pack index searches.

if proxy_config:
https_proxy = proxy_config.get("https_proxy", None)
http_proxy = proxy_config.get("http_proxy", None)
ca_bundle_path = proxy_config.get("proxy_ca_bundle_path", None)
if https_proxy:
proxies_dict["https"] = https_proxy
verify = ca_bundle_path or True
if http_proxy:
proxies_dict["http"] = http_proxy

This makes obvious sense because if you're behind a proxy you want to be able to search the st2 Exchange Index. The issue comes in when you have a local index like I do. The local index is hosted on our controller node and the proxy doesn't have access to this URL due to FW rules. Proxy isn't local to the datacenter either. So because no_proxy isn't respected in an index search my custom index is unreachable.

To further complicate things, I also search the Exchange Index, meaning I have specified both internet based and a local based index. The internet based obviously needs to go though the proxy, but the local based one can't.

The only real way to by pass this at this point is to specify our internal git URL directly and the pack will install since its not searching the index.

Not sure if anyone else has this issue, it might be extremely specific to me with our locked down environment.

STACKSTORM VERSION

Paste the output of st2 --version:
st2 3.6.0, on Python 3.6.8

OS, environment, install method

3 node HA cluster: 2 nodes, 1 controller all three behind a load balancer

@minsis
Copy link
Contributor Author

minsis commented Dec 9, 2021

Actually now that I'm thinking about it the whole point of no_proxy is to do exceptions. So I guess I dont see a reason why the index search also couldn't do no_proxy as well.

@minsis
Copy link
Contributor Author

minsis commented Dec 13, 2021

#5500

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

Successfully merging a pull request may close this issue.

1 participant