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

(#8465) Support SSL in HTTP report via HttpPool #445

Merged
merged 3 commits into from
Feb 2, 2012
Merged

(#8465) Support SSL in HTTP report via HttpPool #445

merged 3 commits into from
Feb 2, 2012

Conversation

lotheac
Copy link
Contributor

@lotheac lotheac commented Feb 2, 2012

Made http reporting use Puppet::Network::HttpPool as per http://projects.puppetlabs.com/issues/8465#note-7 adding support for SSL with verification.

slippycheeze added a commit that referenced this pull request Feb 2, 2012
(#8465) Support SSL in HTTP report via HttpPool
@slippycheeze slippycheeze merged commit ca50b7c into puppetlabs:master Feb 2, 2012
@trunet
Copy link

trunet commented Mar 12, 2012

I applied this pull request and I'm getting:
Mar 12 15:16:54 puppet puppet-master[18910]: Report processor failed: SSL_connect returned=1 errno=0 state=SSLv3 read server certificate B: certificate verify failed

It runs fine applying the other pull request(#175).

@slippycheeze
Copy link
Contributor

G'day @trunet. Am I understanding correctly here: you manually applied this change to your code, and are seeing a certificate validation problem?

Can you file a ticket at https://projects.puppetlabs.com/projects/puppet with the details of your environment - which Ruby version, which Puppet version, and so on? Generally, we don't support this except where we applied it (eg: in the forthcoming 3.0.0 release, presently in RC), but I would like to fully understand your issue.

melissa pushed a commit to melissa/puppet that referenced this pull request Mar 30, 2018
(PCP-472) Ignore pxp-module-puppet `env` input arg
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 this pull request may close these issues.

3 participants