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

(MODULES-10246) Proxy settings allow empty string #22

Conversation

Dorin-Pleava
Copy link
Contributor

This PR enables the Yum Repo Core to accept an empty string as proxy
setting, to bypass any global proxy settings when accessing the
repository.

@Dorin-Pleava Dorin-Pleava requested a review from a team December 12, 2019 15:15
@Dorin-Pleava Dorin-Pleava force-pushed the MODULES-10246/yum_repo_core_proxy_accepts_empty_string branch from 13d8c88 to 05e93e3 Compare December 13, 2019 14:14
@Dorin-Pleava Dorin-Pleava requested review from luchihoratiu, GabrielNagy, gimmyxd and a team and removed request for a team December 13, 2019 14:14
lib/puppet/type/yumrepo.rb Outdated Show resolved Hide resolved
spec/unit/type/yumrepo_spec.rb Show resolved Hide resolved
@Dorin-Pleava Dorin-Pleava force-pushed the MODULES-10246/yum_repo_core_proxy_accepts_empty_string branch from 05e93e3 to ffd1db6 Compare December 13, 2019 15:27
@Dorin-Pleava Dorin-Pleava force-pushed the MODULES-10246/yum_repo_core_proxy_accepts_empty_string branch from ffd1db6 to c0499f3 Compare December 16, 2019 12:53
@mihaibuzgau mihaibuzgau requested a review from a team December 17, 2019 13:02
This PR enables the Yum Repo Core to accept an empty string as proxy
setting, to bypass any global proxy settings when accessing the
repository.

In case of EL 8, '_none_' will be replaced with '' (empty string).
@mihaibuzgau mihaibuzgau force-pushed the MODULES-10246/yum_repo_core_proxy_accepts_empty_string branch from c0499f3 to 594bb32 Compare December 18, 2019 11:47
@mihaibuzgau mihaibuzgau merged commit 4ec747e into puppetlabs:master Dec 18, 2019
@mihaibuzgau mihaibuzgau added the bug Something isn't working label Dec 18, 2019
@bodgit
Copy link

bodgit commented Dec 18, 2019

I think this change has broken the stahnma/epel module. With version 1.0.5 of this module I now get proxy=absent lines added to each of the EPEL repo files which breaks Yum. If I downgrade this module back down to 1.0.4 I get the correct behaviour.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants