[master] Add proxy option to gitfs and git_pillar#66744
[master] Add proxy option to gitfs and git_pillar#66744dwoz merged 5 commits intosaltstack:masterfrom
Conversation
|
Hi there! Welcome to the Salt Community! Thank you for making your first contribution. We have a lengthy process for issues and PRs. Someone from the Core Team will follow up as soon as possible. In the meantime, here’s some information that may help as you continue your Salt journey.
There are lots of ways to get involved in our community. Every month, there are around a dozen opportunities to meet with other contributors and the Salt Core team and collaborate in real time. The best way to keep track is by subscribing to the Salt Community Events Calendar. |
|
@eisenmann-b1 Need to tick the |
I did not tick it since there are no new tests, which I felt would be necessary for this. |
|
@eisenmann-b1 you |
|
@dmurphy18 ok, I ticked the box. |
5a2bc39 to
bb09062
Compare
|
Congratulations on your first PR being merged! 🎉 |
What does this PR do?
This PR add the options
gitfs_proxy,git_pillar_proxyandwinrepo_proxy, with which a proxy server can be specified to be used for connecting to git repositories. These options can also be set per remote asproxy, overriding the global options.For Pygit2, a warning will be given if
proxyis set and the URL to the git repository does not have anhttpsscheme.This is due to an libgit2/libgit2#5650 in libgit2, the library pygit2 is using.
What issues does this PR fix or reference?
Fixes #30990
Previous Behavior
There is no option to specify the proxy server. The workaround described in #30990 requires changes to code akin to the ones here, or setting environment variables in service files.
New Behavior
A proxy server can now be specified like this:
In this example, the repository at
https://foo.com/bar.gitis fetched through the proxyhttp://proxy.com:8080/.Merge requirements satisfied?
[NOTICE] Bug fixes or features added to Salt require tests.
Commits signed with GPG?
Yes