Skip to content
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

Fixes #5639: Adds proxy options for Katello and CDN settings. #27

Merged
merged 1 commit into from
Jun 19, 2014

Conversation

ehelms
Copy link
Member

@ehelms ehelms commented Jun 9, 2014

No description provided.

$proxy_host = $katello::params::proxy_host,
$proxy_port = $katello::params::proxy_port,
$proxy_username = $katello::params::proxy_username,
$proxy_password = $katello::params::proxy_password
Copy link
Member

Choose a reason for hiding this comment

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

You can use trailing commas nowadays.

@mccun934
Copy link
Contributor

tested this and it worked perfectly once I used the IP address of the proxy and not the hostname.

ACK

ehelms added a commit that referenced this pull request Jun 19, 2014
Fixes #5639: Adds proxy options for Katello and CDN settings.
@ehelms ehelms merged commit 3868116 into theforeman:master Jun 19, 2014
@ehelms ehelms deleted the fixes-5639 branch June 19, 2014 15:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants