Skip to content

support per-repo HTTP proxy for the git provider #576

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Nov 2, 2022

Conversation

bugfood
Copy link

@bugfood bugfood commented Oct 24, 2022

Proxies can be specified either as a single proxy or as one or more per-remote proxies.

Since the proxy config need not be supported for every git command, define a new wrapper function to be used for git operations that need remote access, and then prepend the proxy arguments in that wrapper.

This implementation is better than the previously-documented method of using proxies when:
a. Global configuration is not desired.
b. Initial git clone should/must use a proxy.

The previously-documented method is still available for different use cases; reword the original instructions so that they are specified as an alternative rather than the primary method.

Move all proxy information into its own section, since this has become rather large to be included in general git usage.

Proxies can be specified either as a single proxy or as one or more
per-remote proxies.

Since the proxy config need not be supported for _every_ git command,
define a new wrapper function to be used for git operations that need
remote access, and then prepend the proxy arguments in that wrapper.

This implementation is better than the previously-documented method of
using proxies when:
a. Global configuration is not desired.
b. Initial `git clone` should/must use a proxy.

The previously-documented method is still available for different use
cases; reword the original instructions so that they are specified as an
alternative rather than the primary method.

Move all proxy information into its own section, since this has become
rather large to be included in general git usage.
@bugfood bugfood requested a review from a team as a code owner October 24, 2022 19:58
@puppet-community-rangefinder
Copy link

vcsrepo is a type

Breaking changes to this file WILL impact these 178 modules (exact match):
Breaking changes to this file MAY impact these 66 modules (near match):

This module is declared in 110 of 579 indexed public Puppetfiles.


These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.

Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.

Copy link
Member

@david22swan david22swan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link

@LukasAud LukasAud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for your contribution!

@LukasAud LukasAud merged commit 3d78c5b into puppetlabs:main Nov 2, 2022
@bugfood
Copy link
Author

bugfood commented Nov 2, 2022

You're welcome, and thank you.

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants