Issue #15975 has been updated by Chris Price. Status changed from In Topic Branch Pending Review to Merged - Pending Release
Patrick merged this in 6858f02 ---------------------------------------- Refactor #15975: Make Puppet::Indirector::REST http utility methods accessible to other classes https://projects.puppetlabs.com/issues/15975#change-69516 Author: Chris Price Status: Merged - Pending Release Priority: Normal Assignee: Jeff McCune Category: Target version: 3.0.0 Affected Puppet version: Keywords: Branch: https://github.com/puppetlabs/puppet/pull/1040 There are some useful HTTP methods (http_get, http_post, ..., http_request) in the Puppet::Indirector::REST class. Most noteworthy is the `http_request` method, which provides some useful error handling so that users get less confusing error messages when something goes wrong with their certificates / SSL setup. These methods would be useful in several other places (among them: the HTTP reports processor and several places in the puppetdb termini), but at present it is very difficult (or impossible) to use them without doing a very confusing dance with unnecessary ruby mixins and inheritance from classes in the Indirectory hierarchy. These methods should be refactored out into their own class or module so that they can be used elsewhere. -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
